public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto
@ 2010-10-20  3:17 H.J. Lu
  2010-10-20  8:51 ` Paolo Bonzini
  0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2010-10-20  3:17 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gcc-patches

On Sun, Oct 17, 2010 at 3:07 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> Hi,
>
> This patch enables lto for stage1 if LTO is enabled. OK for trunk?
>
> Thanks.
>
>
> H.J.
> ---
> 2010-10-17  H.J. Lu  <hongjiu.lu@intel.com>
>
>        PR bootstrap/45954
>        * configure.ac: Enable lto for stage1 if LTO is enabled.
>        * configure: Regenerated.
>
> diff --git a/configure b/configure
> index b1fbfaa..1397c11 100755
> --- a/configure
> +++ b/configure
> @@ -6711,7 +6711,11 @@ fi
>
>
>  # By default, C is the only stage 1 language.
> -stage1_languages=,c,
> +if test "$enable_lto" = "yes" ; then
> +  stage1_languages=,c,lto,
> +else
> +  stage1_languages=,c,
> +fi
>
>  # Target libraries that we bootstrap.
>  bootstrap_target_libs=,target-libgcc,
> @@ -6905,7 +6909,12 @@ if test "${enable_stage1_languages+set}" = set; then :
>   enableval=$enable_stage1_languages; case ,${enable_stage1_languages}, in
>     ,no,|,,)
>       # Set it to something that will have no effect in the loop below
> -      enable_stage1_languages=c ;;
> +      if test "$enable_lto" = "yes" ; then
> +        enable_stage1_languages=c,lto
> +      else
> +        enable_stage1_languages=c
> +      fi
> +      ;;
>     ,yes,)
>       enable_stage1_languages=`echo $new_enable_languages | \
>        sed -e "s/^,//" -e "s/,$//" ` ;;
> diff --git a/configure.ac b/configure.ac
> index bf5fe3a..f4521bd 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1817,7 +1817,11 @@ fi],[if test x"$default_enable_lto" = x"yes" ; then
>
>
>  # By default, C is the only stage 1 language.
> -stage1_languages=,c,
> +if test "$enable_lto" = "yes" ; then
> +  stage1_languages=,c,lto,
> +else
> +  stage1_languages=,c,
> +fi
>
>  # Target libraries that we bootstrap.
>  bootstrap_target_libs=,target-libgcc,
> @@ -2012,7 +2016,12 @@ if test -d ${srcdir}/gcc; then
>   [case ,${enable_stage1_languages}, in
>     ,no,|,,)
>       # Set it to something that will have no effect in the loop below
> -      enable_stage1_languages=c ;;
> +      if test "$enable_lto" = "yes" ; then
> +        enable_stage1_languages=c,lto
> +      else
> +        enable_stage1_languages=c
> +      fi
> +      ;;
>     ,yes,)
>       enable_stage1_languages=`echo $new_enable_languages | \
>        sed -e "s/^,//" -e "s/,$//" ` ;;
>

PING

-- 
H.J.

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

* Re: PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto
  2010-10-20  3:17 PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto H.J. Lu
@ 2010-10-20  8:51 ` Paolo Bonzini
  2010-10-20 14:20   ` H.J. Lu
  0 siblings, 1 reply; 4+ messages in thread
From: Paolo Bonzini @ 2010-10-20  8:51 UTC (permalink / raw)
  To: H.J. Lu; +Cc: gcc-patches

On 10/20/2010 01:52 AM, H.J. Lu wrote:
> On Sun, Oct 17, 2010 at 3:07 PM, H.J. Lu<hongjiu.lu@intel.com>  wrote:
>> Hi,
>>
>> This patch enables lto for stage1 if LTO is enabled. OK for trunk?

Maybe

boot_language=$enable_lto

in gcc/config/lto/config-lang.in is enough?

Paolo

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

* Re: PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto
  2010-10-20  8:51 ` Paolo Bonzini
@ 2010-10-20 14:20   ` H.J. Lu
  2010-10-20 14:40     ` Paolo Bonzini
  0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2010-10-20 14:20 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gcc-patches

On Tue, Oct 19, 2010 at 11:26 PM, Paolo Bonzini <bonzini@gnu.org> wrote:
> On 10/20/2010 01:52 AM, H.J. Lu wrote:
>>
>> On Sun, Oct 17, 2010 at 3:07 PM, H.J. Lu<hongjiu.lu@intel.com>  wrote:
>>>
>>> Hi,
>>>
>>> This patch enables lto for stage1 if LTO is enabled. OK for trunk?
>
> Maybe
>
> boot_language=$enable_lto
>
> in gcc/config/lto/config-lang.in is enough?
>

It works.  OK for trunk?

Thanks.


-- 
H.J.
---
2010-10-20  H.J. Lu  <hongjiu.lu@intel.com>

	PR bootstrap/45954
	* config-lang.in (boot_language): Set to $enable_lto.

diff --git a/gcc/lto/config-lang.in b/gcc/lto/config-lang.in
index aa84db1..72ed2dd 100644
--- a/gcc/lto/config-lang.in
+++ b/gcc/lto/config-lang.in
@@ -30,3 +30,6 @@ gtfiles="\$(srcdir)/lto/lto-tree.h
\$(srcdir)/lto/lto-lang.c \$(srcdir)/lto/lto.
 # enabling this "language". To enable LTO functionality, use
 # --enable-lto when configuring the compiler.
 build_by_default=no
+
+# Add LTO to boot language if it is enabled.
+boot_language=$enable_lto

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

* Re: PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto
  2010-10-20 14:20   ` H.J. Lu
@ 2010-10-20 14:40     ` Paolo Bonzini
  0 siblings, 0 replies; 4+ messages in thread
From: Paolo Bonzini @ 2010-10-20 14:40 UTC (permalink / raw)
  To: H.J. Lu; +Cc: gcc-patches

On 10/20/2010 02:23 PM, H.J. Lu wrote:
> On Tue, Oct 19, 2010 at 11:26 PM, Paolo Bonzini<bonzini@gnu.org>  wrote:
>> On 10/20/2010 01:52 AM, H.J. Lu wrote:
>>>
>>> On Sun, Oct 17, 2010 at 3:07 PM, H.J. Lu<hongjiu.lu@intel.com>    wrote:
>>>>
>>>> Hi,
>>>>
>>>> This patch enables lto for stage1 if LTO is enabled. OK for trunk?
>>
>> Maybe
>>
>> boot_language=$enable_lto
>>
>> in gcc/config/lto/config-lang.in is enough?
>>
>
> It works.  OK for trunk?

Yes.

Paolo

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

end of thread, other threads:[~2010-10-20 12:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-20  3:17 PING: PATCH: PR bootstrap/45954: LTO isn't enabled in stage1 cc1 with --with-build-config=bootstrap-lto H.J. Lu
2010-10-20  8:51 ` Paolo Bonzini
2010-10-20 14:20   ` H.J. Lu
2010-10-20 14:40     ` Paolo Bonzini

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).