Update to compile with the Intel compiler.

.gitignore

@@ -21,3 +21,4 @@ stdout
 *.orig
 *.rej
 *.old
+TAGS

ChangeLog

+2008-12-11  Guillaume SADEGH  <sadegh@lrde.epita.fr>
+
+	Update to compile with the Intel compiler.
+
+  	* m4/intel.m4: New file.
+	* configure.ac: Update.
+	* src/tgbaalgos/emptiness.cc (tgba_run): Modify s.s->clone() in
+	i->s->clone().
+	* src/misc/optionmap.hh, src/misc/optionmap.cc: Remove the extra
+	`;' after the namespace.
+        * src/tgbaalgos/tau03opt.cc
+	(tau03_opt_search::add_new_state): Remove unreferenced method.
+        * src/tgbaalgos/ltl2tgba_fm.cc
+	(translate_dict::dump): Remove unreferenced method.
+        * src/tgbaalgos/lbtt.cc
+	(acceptance_cond_splitter::count): Remove unreferenced method.
+        * src/tgba/tgbabddconcreteproduct.cc
+	(tgba_bdd_product_factory::get_dict): Remove unreferenced method.
+        * src/ltlvisit/syntimpl.cc
+        (eventual_universal_visitor::recurse): Remove unreferenced method.
+	* src/tgbaalgos/reductgba_sim.cc: Reindent.
+
 2008-12-11  Alexandre Duret-Lutz  <adl@lrde.epita.fr>
 
 	Compute more statistics about SCCs.

configure.ac

@@ -51,6 +51,7 @@ AC_CHECK_FUNCS([srand48 drand48])
 AC_LIBTOOL_WIN32_DLL
 AC_PROG_LIBTOOL
 
+spot_INTEL
 CF_GXX_WARNINGS
 adl_ENABLE_DEBUG
 ad_GCC_OPTIM

m4/intel.m4

+
+
+dnl Adapted from the predefined _AC_LANG_COMPILER_GNU.
+m4_define([_AC_LANG_COMPILER_INTEL],
+[AC_CACHE_CHECK([whether we are using the INTEL _AC_LANG compiler],
+		[ac_cv_[]_AC_LANG_ABBREV[]_compiler_intel],
+[
+_AC_COMPILE_IFELSE([AC_LANG_PROGRAM([], [[#ifndef __INTEL_COMPILER
+       choke me
+#endif
+]])],
+		   [ac_compiler_intel=yes],
+		   [ac_compiler_intel=no])
+ac_cv_[]_AC_LANG_ABBREV[]_compiler_intel=$ac_compiler_intel
+])])# _AC_LANG_COMPILER_INTEL
+
+dnl The list of warnings which must be disabled for Spot.
+m4_define([_INTEL_IGNORE_WARNINGS],
+[ 69   dnl Warn when an enum value is used as an int, without any
+       dnl explicit cast.
+  279  dnl Warn when a constant expression is used in a control statement.
+  654  dnl Warn when a child does not overload all virtual members of his
+       dnl parents.
+  913  dnl Warn when multibyte character are used (for example "Büchi").
+  1125 dnl Warn when a child member hides a parent member.
+]) # _INTEL_IGNORE_WARNINGS
+
+dnl Add extra flags when icpc is used.
+AC_DEFUN([spot_INTEL],
+[_AC_LANG_COMPILER_INTEL
+AC_CACHE_CHECK([to add extra CXXFLAGS for the INTEL C++ compiler],
+		[ac_cv_intel_cxxflags],
+[ dnl
+  if test x"$ac_compiler_intel" = x"yes"; then
+    disabled_warnings=''
+    for warning in _INTEL_IGNORE_WARNINGS; do
+      disabled_warnings="$disabled_warnings,$warning"
+    done
+
+    # remove the extra "," and extra whitespaces.
+    disabled_warnings=`echo "$disabled_warnings" | sed "s/^,//;s/[[:space:]]//g"`
+
+    INTEL_CXXFLAGS="-w1 -Werror -wd${disabled_warnings}"
+
+    # Even if the icpc preprocessor defines __GNUC__, it is not a GNU compiler.
+    GXX=
+    CXXFLAGS="$CXXFLAGS $INTEL_CXXFLAGS"
+
+    [ac_cv_intel_cxxflags="$INTEL_CXXFLAGS"]
+  else
+    [ac_cv_intel_cxxflags="no extra flags"]
+  fi])
+  AC_SUBST([INTEL_CXXFLAGS])
+]) # spot_INTEL

src/ltlvisit/syntimpl.cc

@@ -301,16 +301,6 @@ namespace spot
 	    }
 	}
 
-	bool
-	recurse(const formula* f1, const formula* f2)
-	{
-	  if (f1 == f2)
-	    return true;
-	  inf_right_recurse_visitor v(f2);
-	  const_cast<formula*>(f1)->accept(v);
-	  return v.result();
-	}
-
       protected:
 	bool result_; /* true if f < f1, false otherwise. */
 	const formula* f;

src/misc/optionmap.cc

@@ -157,4 +157,4 @@ namespace spot
       os << "\"" << it->first << "\" = " << it->second << std::endl;
     return os;
   }
-};
+}

src/misc/optionmap.hh

@@ -82,6 +82,6 @@ namespace spot
   private:
     std::map<std::string, int> options_;
   };
-};
+}
 
 #endif // SPOT_MISC_OPTIONMAP_HH

src/tgba/tgbabddconcreteproduct.cc

@@ -56,12 +56,6 @@ namespace spot
 	return data_;
       }
 
-      bdd_dict*
-      get_dict() const
-      {
-	return dict_;
-      }
-
       bdd
       get_init_state() const
       {

src/tgbaalgos/emptiness.cc

@@ -50,13 +50,13 @@ namespace spot
     for (steps::const_iterator i = run.prefix.begin();
 	 i != run.prefix.end(); ++i)
       {
-	step s = { s.s->clone(), i->label, i->acc };
+	step s = { i->s->clone(), i->label, i->acc };
 	prefix.push_back(s);
       }
     for (steps::const_iterator i = run.cycle.begin();
 	 i != run.cycle.end(); ++i)
       {
-	step s = { s.s->clone(), i->label, i->acc };
+	step s = { i->s->clone(), i->label, i->acc };
 	cycle.push_back(s);
       }
   }

src/tgbaalgos/lbtt.cc

@@ -62,12 +62,6 @@ namespace spot
 	return os;
       }
 
-      unsigned
-      count() const
-      {
-	return sm.size();
-      }
-
     private:
       typedef std::map<bdd, unsigned, bdd_less_than> split_map;
       split_map sm;

src/tgbaalgos/ltl2tgba_fm.cc

@@ -117,21 +117,6 @@ namespace spot
 	return num;
       }
 
-      std::ostream&
-      dump(std::ostream& os) const
-      {
-	fv_map::const_iterator fi;
-	os << "Next Variables:" << std::endl;
-	for (fi = next_map.begin(); fi != next_map.end(); ++fi)
-	{
-	  os << "  " << fi->second << ": Next[";
-	  fi->first->to_string(os) << "]" << std::endl;
-	}
-	os << "Shared Dict:" << std::endl;
-	dict->dump(os);
-	return os;
-      }
-
       formula*
       var_to_formula(int var) const
       {

src/tgbaalgos/reductgba_sim.cc

@@ -276,16 +276,16 @@ namespace spot
 
   void
   parity_game_graph::process_state(const state* s,
-				    int ,
-				    tgba_succ_iterator*)
+				   int,
+				   tgba_succ_iterator*)
   {
     tgba_state_.push_back(s);
   }
 
   void
-  parity_game_graph::process_link(int ,
-				   int ,
-				   const tgba_succ_iterator*)
+  parity_game_graph::process_link(int,
+				  int,
+				  const tgba_succ_iterator*)
   {
   }
 

src/tgbaalgos/tau03opt.cc

@@ -530,13 +530,6 @@ namespace spot
                               &(ic->second.first), &(ic->second.second));
         }
 
-      void add_new_state(const state* s, color c)
-        {
-          assert(hc.find(s)==hc.end() && h.find(s)==h.end());
-          assert(c != CYAN);
-          h.insert(std::make_pair(s, std::make_pair(c, bddfalse)));
-        }
-
       void add_new_state(const state* s, color c, const weight& w)
         {
           assert(hc.find(s)==hc.end() && h.find(s)==h.end());