[patches] Re: Recognize cpu varaints take III

Eric Christopher echristo@redhat.com
Fri Dec 14 12:00:00 GMT 2001


On Fri, 2001-12-14 at 11:44, DJ Delorie wrote:
> 
> > Oops, no room for in MASK - we do have more than 32bits used.
> > Was this ever hit in some other backend? What is the most convenient
> > sollution to this situation?
> 
> MIPS hits it on a regular basis (echristo cc'd).  You can use
> TARGET_OPTIONS for some of the options, if they don't need the -mno-
> variant.

Right.  I've switched to using -march/-mtune for all cpu variants and
only using masks for turning on and off options. Parsing all of the
variants is done in mips.c.

Question: You're using MASK_MMX and MASK_MMX_SET. From a quick glance I
couldn't tell what the difference was (and I mean quick). The *_SET
appear to be taking up a lot of bits...

> 
> I've been thinking of expanding TARGET_OPTIONS to support an optional
> "value" field, which would help with this situation.  Maybe I'll work
> on that proposal today.

*shrug* You can just use whatever variable as a boolean here...

-eric

-- 
Fridays are not "pants optional"



More information about the Gcc-patches mailing list