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

On Tue, May 04, 2004 at 11:14:18AM -0700, Richard Henderson wrote:
> On Tue, May 04, 2004 at 01:24:39PM -0400, Aldy Hernandez wrote:
> > As discussed on irc, having a 2 dimensional array requires us to
> > define the size ahead of time, and NUM_MACHINE_MODES is defined in
> > insn-modes.h.
> 
> False.  For the declaration in a header, you can leave the outer
> dimension of an array unspecified:
> 
>   extern bool rs6000_hard_regno_mode_ok_p[][FIRST_PSEUDO_REGISTER];

Ah!!! I had it the other way around (which didn't work):

      rs6000_hard_regno_mode_ok_p[FIRST_PSEUDO_REGISTER][]

Fixed.

> Also, why use an int to store a 1 bit value?

David, what was the deal with using (or not using) bool?

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