This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: patch: pre-compute HARD_REGNO_MODE_OK on ppc

Richard:

> > 	I would prefer bool, but I was concerned if that was safe based on
> > the original documentation of the macro.  Based on the current documentat
> > ("C expression that is nonzero") and its uses within the compiler, please
> > use an array of bool.
> 
> In that case... Fixed.
> 
> Testing is still going.  I will have testresults done by morning (slow
> box).

In file included from ./tm.h:4,
                 from /home/cygnus/aldyh/source/untainted-sources/gcc/gcc/crtstuff.c:64:
/home/cygnus/aldyh/source/untainted-sources/gcc/gcc/config/rs6000/rs6000.h:1035: error: syntax error before "rs6000_hard_regno_mode_ok_p"

crtstuff.c includes tm.h but does not include anything providing
`bool'.  Should we fix crtstuff, or go back to my int ways?

Aldy
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]