[patch] Fix bootstrap on i386-pc-solaris2.10

Rainer Orth ro@techfak.uni-bielefeld.de
Fri Jul 25 12:25:00 GMT 2008


Art Haas <ahaas@impactweather.com> writes:

> I needed the trivial one-liner below to have GCC build again after the
> patch on 7/23 landed which fiddled with the REGISTER_TARGET_PRAGMAS
> macro. While the patch makes the compiler bootstrap succeed, whether or
> not it is the correct fix will require someone with more expertise to
> determine.
> 
> Art Haas
> 
> diff --git a/gcc/config/sol2.h b/gcc/config/sol2.h
> index 512d0ff..5247bf2 100644
> --- a/gcc/config/sol2.h
> +++ b/gcc/config/sol2.h
> @@ -234,6 +234,7 @@ __enable_execute_stack (void *addr)					\
>    while (0)
>  
>  /* Register the Solaris-specific #pragma directives.  */
> +#undef REGISTER_TARGET_PRAGMAS
>  #define REGISTER_TARGET_PRAGMAS() solaris_register_pragmas ()
>  
>  extern GTY(()) tree solaris_pending_aligns;

I think the patch is wrong as it loses the generic i386
REGISTER_TARGET_PRAGMAS.  The correct solution is to use
REGISTER_SUBTARGET_PRAGMAS as in darwin.h.  I'll submit a patch shortly.

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Faculty of Technology, Bielefeld University



More information about the Gcc-patches mailing list