On Tue, Apr 21, 2015 at 04:24:44PM +0100, Trevor Saunders wrote:
> On Tue, Apr 21, 2015 at 04:14:01PM +0200, Richard Biener wrote:
> > On Tue, Apr 21, 2015 at 3:24 PM,  <tbsaunde+...@tbsaunde.org> wrote:
> > > From: Trevor Saunders <tbsaunde+...@tbsaunde.org>
> > >
> > > gcc/ChangeLog:
> > >
> > > 2015-04-21  Trevor Saunders  <tbsaunde+...@tbsaunde.org>
> > >
> > >         * conditions.h: Define macros even if HAVE_cc0 is undefined.
> > >         * emit-rtl.c: Define functions even if HAVE_cc0 is undefined.
> > >         * final.c: Likewise.
> > >         * jump.c: Likewise.
> > >         * recog.c: Likewise.
> > >         * recog.h: Declare functions even when HAVE_cc0 is undefined.
> > >         * sched-deps.c (sched_analyze_2): Always compile case for cc0.

If I've counted right after the git bisect, this patch seems to break
the ARM buildi (arm-none-linux-gnueabihf):

  In file included from insn-output.c:40:0:
  ..../gcc-src/gcc/conditions.h:112:0: error: "CC_STATUS_INIT" redefined 
[-Werror]
   #define CC_STATUS_INIT  \
   ^
  In file included from tm.h:35:0,
                   from insn-output.c:7:
  ..../gcc-src/gcc/config/arm/arm.h:2159:0: note: this is the location of the 
previous definition
   #define CC_STATUS_INIT \
   ^

I guess the conditions.h definition wants wrapping in #ifndef - though a
quick grep suggests that ARM is the only target defining CC_STATUS_INIT
so lets CC the ARM maintainers and see what their preference is...

Thanks,
James

Reply via email to