public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [cxx-conversion] Switch default to build in C++
@ 2012-04-11 16:37 Diego Novillo
  2012-04-12  8:31 ` Richard Guenther
  0 siblings, 1 reply; 3+ messages in thread
From: Diego Novillo @ 2012-04-11 16:37 UTC (permalink / raw)
  To: gcc-patches

This patch makes the branch to build in C++ mode by default.

Tested on x86_64.  Committed.


ChangeLog.cxx-conversion
	* configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
	* configure: Regenerate.

gcc/ChangeLog.cxx-conversion
	* configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
	* configure: Regenerate.

libcpp/ChangeLog.cxx-conversion
	* configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
	* configure: Regenerate.

Index: configure.ac
===================================================================
--- configure.ac	(revision 186326)
+++ configure.ac	(working copy)
@@ -1191,7 +1191,7 @@
 [AS_HELP_STRING([--enable-build-with-cxx],
 		[build with C++ compiler instead of C compiler])],
 ENABLE_BUILD_WITH_CXX=$enableval,
-ENABLE_BUILD_WITH_CXX=no)
+ENABLE_BUILD_WITH_CXX=yes)
 
 # Build stage1 with C and build stages 2 and 3 with C++.
 AC_ARG_ENABLE(build-poststage1-with-cxx,
Index: gcc/configure.ac
===================================================================
--- gcc/configure.ac	(revision 186326)
+++ gcc/configure.ac	(working copy)
@@ -608,7 +608,7 @@
 [AS_HELP_STRING([--enable-build-with-cxx],
 		[build with C++ compiler instead of C compiler])],
 ENABLE_BUILD_WITH_CXX=$enableval,
-ENABLE_BUILD_WITH_CXX=no)
+ENABLE_BUILD_WITH_CXX=yes)
 AC_SUBST(ENABLE_BUILD_WITH_CXX)
 if test "$ENABLE_BUILD_WITH_CXX" = "yes"; then
   AC_DEFINE(ENABLE_BUILD_WITH_CXX, 1,
Index: libcpp/configure.ac
===================================================================
--- libcpp/configure.ac	(revision 186326)
+++ libcpp/configure.ac	(working copy)
@@ -22,7 +22,7 @@
 AC_ARG_ENABLE(build-with-cxx,
 [  --enable-build-with-cxx build with C++ compiler instead of C compiler],
 ENABLE_BUILD_WITH_CXX=$enableval,
-ENABLE_BUILD_WITH_CXX=no)
+ENABLE_BUILD_WITH_CXX=yes)
 AC_SUBST(ENABLE_BUILD_WITH_CXX)
 
 MISSING=`cd $ac_aux_dir && ${PWDCMD-pwd}`/missing

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [cxx-conversion] Switch default to build in C++
  2012-04-11 16:37 [cxx-conversion] Switch default to build in C++ Diego Novillo
@ 2012-04-12  8:31 ` Richard Guenther
  2012-04-12 12:16   ` Diego Novillo
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Guenther @ 2012-04-12  8:31 UTC (permalink / raw)
  To: Diego Novillo; +Cc: gcc-patches

On Wed, Apr 11, 2012 at 6:37 PM, Diego Novillo <dnovillo@google.com> wrote:
> This patch makes the branch to build in C++ mode by default.
>
> Tested on x86_64.  Committed.

Note that for merging the branch the support to build with a C host compiler
should be removed (and docs updated).

Richard.

>
> ChangeLog.cxx-conversion
>        * configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
>        * configure: Regenerate.
>
> gcc/ChangeLog.cxx-conversion
>        * configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
>        * configure: Regenerate.
>
> libcpp/ChangeLog.cxx-conversion
>        * configure.ac (ENABLE_BUILD_WITH_CXX): Default to 'yes'.
>        * configure: Regenerate.
>
> Index: configure.ac
> ===================================================================
> --- configure.ac        (revision 186326)
> +++ configure.ac        (working copy)
> @@ -1191,7 +1191,7 @@
>  [AS_HELP_STRING([--enable-build-with-cxx],
>                [build with C++ compiler instead of C compiler])],
>  ENABLE_BUILD_WITH_CXX=$enableval,
> -ENABLE_BUILD_WITH_CXX=no)
> +ENABLE_BUILD_WITH_CXX=yes)
>
>  # Build stage1 with C and build stages 2 and 3 with C++.
>  AC_ARG_ENABLE(build-poststage1-with-cxx,
> Index: gcc/configure.ac
> ===================================================================
> --- gcc/configure.ac    (revision 186326)
> +++ gcc/configure.ac    (working copy)
> @@ -608,7 +608,7 @@
>  [AS_HELP_STRING([--enable-build-with-cxx],
>                [build with C++ compiler instead of C compiler])],
>  ENABLE_BUILD_WITH_CXX=$enableval,
> -ENABLE_BUILD_WITH_CXX=no)
> +ENABLE_BUILD_WITH_CXX=yes)
>  AC_SUBST(ENABLE_BUILD_WITH_CXX)
>  if test "$ENABLE_BUILD_WITH_CXX" = "yes"; then
>   AC_DEFINE(ENABLE_BUILD_WITH_CXX, 1,
> Index: libcpp/configure.ac
> ===================================================================
> --- libcpp/configure.ac (revision 186326)
> +++ libcpp/configure.ac (working copy)
> @@ -22,7 +22,7 @@
>  AC_ARG_ENABLE(build-with-cxx,
>  [  --enable-build-with-cxx build with C++ compiler instead of C compiler],
>  ENABLE_BUILD_WITH_CXX=$enableval,
> -ENABLE_BUILD_WITH_CXX=no)
> +ENABLE_BUILD_WITH_CXX=yes)
>  AC_SUBST(ENABLE_BUILD_WITH_CXX)
>
>  MISSING=`cd $ac_aux_dir && ${PWDCMD-pwd}`/missing

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [cxx-conversion] Switch default to build in C++
  2012-04-12  8:31 ` Richard Guenther
@ 2012-04-12 12:16   ` Diego Novillo
  0 siblings, 0 replies; 3+ messages in thread
From: Diego Novillo @ 2012-04-12 12:16 UTC (permalink / raw)
  To: Richard Guenther; +Cc: gcc-patches

On 4/12/12 4:31 AM, Richard Guenther wrote:
> On Wed, Apr 11, 2012 at 6:37 PM, Diego Novillo<dnovillo@google.com>  wrote:
>> This patch makes the branch to build in C++ mode by default.
>>
>> Tested on x86_64.  Committed.
>
> Note that for merging the branch the support to build with a C host compiler
> should be removed (and docs updated).

Thanks.  Yes, I already had that in mind.


Diego.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-04-12 12:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-11 16:37 [cxx-conversion] Switch default to build in C++ Diego Novillo
2012-04-12  8:31 ` Richard Guenther
2012-04-12 12:16   ` Diego Novillo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).