On 12-06-20 10:10 , Steven Bosscher wrote:

-# By default, C is the only stage 1 language.
+# By default, C and C++ are the only stage 1 languages.
  stage1_languages=,c,

So shouldn't you add c++ here?

That was a bad change on the comment. We only need C for stage1. Thanks for spotting it.

-  # If bootstrapping, then using --enable-build-with-cxx or
-  # --enable-build-poststage1-with-cxx requires enabling C++.
-  case 
",$enable_languages,:,$ENABLE_BUILD_WITH_CXX,$ENABLE_BUILD_POSTSTAGE1_WITH_CXX,:$enable_bootstrap"
 in
-    *,c++,*:*:*) ;;
-    *:*,yes,*:yes)
+  # If bootstrapping, C++ must be enabled.

Hmn, perhaps I misunderstand, but shouldn't C++ also be enabled if not
bootstrapping?

It's only needed if we are building C++ code. Everything else uses the host compiler.


You can also remove the lang_requires_boot_languages machinery again.
It is only used by Go to enable c++ for bootstrapping the Go front
end, but with c++ enabled by default, there is no need for this hack
for Go anymore.

Good point.  I'll send a separate patch for that.


Diego.

Reply via email to