This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix Solaris 9/x86 bootstrap
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Rainer Orth <ro at cebitec dot uni-bielefeld dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 21 Aug 2012 10:55:30 +0200
- Subject: Re: Fix Solaris 9/x86 bootstrap
- References: <yddipcck4n6.fsf@manam.CeBiTec.Uni-Bielefeld.DE>
On Tue, Aug 21, 2012 at 10:53 AM, Rainer Orth
<ro@cebitec.uni-bielefeld.de> wrote:
> Solaris 9/x86 bootstrap was broken after the cxx-conversion merge:
>
> In file included from /vol/gcc/src/hg/trunk/local/gcc/gengtype.c:957:
> /vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected identifier before n
> umeric constant
> /vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected '}' before numeric
> constant
>
> This happens since g++, unlike gcc, defines __EXTENSIONS__, which
> exposes the equivalent of
>
> #define PC 14
>
> Initially I tried to avoid this by having gengtype.c include rtl.h,
> which already has the #undef, but this produced so much fallout that I
> decided it's better to just replicate it here.
>
> The patch allowed an i386-pc-solaris2.9 bootstrap to finish. I think
> this counts as obvious unless someone prefers the rtl.h route
> nonetheless.
>
> Ok for mainline?
Doesn't that belong in system.h instead? And removed from rtl.h?
Thanks,
Richard.
> Rainer
>
>
> 2012-08-20 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
>
> * gengtype.c (PC): Undef.
>
>
>
> --
> -----------------------------------------------------------------------------
> Rainer Orth, Center for Biotechnology, Bielefeld University
>