On Tue, Aug 07, 2007 at 07:38:45PM +0000, 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.gcc
> +++ b/gcc/config.gcc
> @@ -1265,7 +1265,7 @@ i[34567]86-*-solaris2*)
>       *-*-solaris2.1[0-9]*)
>               tm_file="${tm_file} i386/x86-64.h i386/sol2-10.h"
>               tm_defines="${tm_defines} TARGET_BI_ARCH=1"
> -             tmake_file="$tmake_file i386/t-crtstuff i386/t-sol2-10"
> +             tmake_file="$tmake_file i386/t-sol2-10"
>               need_64bit_hwint=yes
>               # FIXME: -m64 for i[34567]86-*-* should be allowed just
>               # like -m32 for x86_64-*-*.
> 
> The build fails with a variety of link errors when linking libgcc:
> 

CRTSTUFF_T_CFLAGS in config/i386/t-sol2 should override the one
in i386/t-crtstuff anyway. Why didn't it happen? If it does
override the correct CRTSTUFF_T_CFLAGS, we need this patch. Othewise,
we need something different.


H.J.
----
2007-08-07  H.J. Lu  <[EMAIL PROTECTED]>

        * config/i386/t-sol2 (CRTSTUFF_T_CFLAGS): Use += instead of =.

--- gcc/config/i386/t-sol2.sol2 2007-08-07 15:27:10.000000000 -0700
+++ gcc/config/i386/t-sol2      2007-08-07 15:26:17.000000000 -0700
@@ -30,5 +30,5 @@ $(T)crtn.o: $(srcdir)/config/i386/sol2-c
 # the call & alignment statement, but before we switch back to the
 # .text section.
 
-CRTSTUFF_T_CFLAGS = -fPIC -O2
+CRTSTUFF_T_CFLAGS += -fPIC -O2
 TARGET_LIBGCC2_CFLAGS = -fPIC

Reply via email to