Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread Paolo Bonzini
Making the change above results in a successful build. Quite a difference what a '+' can make. This patch works fine on Linux/x86 and Linux/x86-64. Thanks! Paolo

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread H.J. Lu
On Wed, Aug 08, 2007 at 09:45:00AM -0500, Arthur Haas wrote: > > > -Original Message- > > [ ... snip ... ] > > > > > CRTSTUFF_T_CFLAGS_S = $(CRTSTUFF_T_CFLAGS) -fPIC > > > > > > How do you use +=? > > > > Sorry, I was looking at an old checkout. > > > > Could you simply test patching c

RE: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread Arthur Haas
> -Original Message- > [ ... snip ... ] > > > CRTSTUFF_T_CFLAGS_S = $(CRTSTUFF_T_CFLAGS) -fPIC > > > > How do you use +=? > > Sorry, I was looking at an old checkout. > > Could you simply test patching config/i386/t-crtstuff like this: > > -CRTSTUFF_T_CFLAGS = -fno-omit-frame-pointer

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread Paolo Bonzini
CRTSTUFF_T_CFLAGS_S = $(CRTSTUFF_T_CFLAGS) -fPIC How do you use +=? Sorry, I was looking at an old checkout. Could you simply test patching config/i386/t-crtstuff like this: -CRTSTUFF_T_CFLAGS = -fno-omit-frame-pointer -fno-asynchronous-unwind-tables +CRTSTUFF_T_CFLAGS += -fno-omit-frame-po

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread H.J. Lu
On Wed, Aug 08, 2007 at 10:06:03AM +0200, Paolo Bonzini wrote: > > >That is what I missed. Thanks. > > > >This patch should work. > > Sure, but it makes the mess even worse. You should instead: > > - in config/t-svr4, change the two CFLAGS assignments to use += There is CRTSTUFF_T_CFLAGS_S =

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-08 Thread Paolo Bonzini
That is what I missed. Thanks. This patch should work. Sure, but it makes the mess even worse. You should instead: - in config/t-svr4, change the two CFLAGS assignments to use += - in config/i386/t-sol2, change the CRTSTUFF_T_CFLAGS assignment to "+= -O2" and remove the TARGET_LIBGCC2_CFLA

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-07 Thread H.J. Lu
On Tue, Aug 07, 2007 at 06:41:33PM -0400, Daniel Jacobowitz wrote: > On Tue, Aug 07, 2007 at 03:34:56PM -0700, H.J. Lu wrote: > > > - tmake_file="$tmake_file i386/t-crtstuff i386/t-sol2-10" > > > + tmake_file="$tmake_file i386/t-sol2-10" > > > CRTSTUFF_T_CFLAGS in config/i386/t-sol

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-07 Thread Daniel Jacobowitz
On Tue, Aug 07, 2007 at 03:34:56PM -0700, H.J. Lu wrote: > > - tmake_file="$tmake_file i386/t-crtstuff i386/t-sol2-10" > > + tmake_file="$tmake_file i386/t-sol2-10" > CRTSTUFF_T_CFLAGS in config/i386/t-sol2 should override the one > in i386/t-crtstuff anyway. Why didn't it happ

Re: Bootstrap failed on i386-pc-solaris2.10

2007-08-07 Thread H.J. Lu
On Tue, Aug 07, 2007 at 07:38:45PM +, Art Haas wrote: > Hi. > > The patch for PR 31868 broke my builds on this machine. I had to back > out the change to 'config.gnu' for my build to succeed. The resulting > diff is below: > > diff --git a/gcc/config.gcc b/gcc/config.gcc > --- a/gcc/config.gc

Bootstrap failed on i386-pc-solaris2.10

2007-08-07 Thread Art Haas
Hi. The patch for PR 31868 broke my builds on this machine. I had to back out the change to 'config.gnu' for my build to succeed. The resulting diff is below: diff --git a/gcc/config.gcc b/gcc/config.gcc --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -1265,7 +1265,7 @@ i[34567]86-*-solaris2*)