On Mon, Jun 17, 2019 at 08:04:42PM -0400, Michael Meissner wrote: > --- gcc/config/rs6000/rs6000.md (revision 272270) > +++ gcc/config/rs6000/rs6000.md (working copy) > @@ -267,7 +267,9 @@ (define_attr "cpu" > (const (symbol_ref "(enum attr_cpu) rs6000_tune"))) > > ;; The ISA we implement. > -(define_attr "isa" "any,p5,p6,p7,p7v,p8v,p9v,p9kf,p9tf" (const_string "any")) > +(define_attr "isa" > + "any,p5,p6,p7,p7v,p8v,p9v,p9kf,p9tf,prefix" > + (const_string "any"))
Don't break the line unnecessarily. "prefix" is not a good name. Maybe use "futp" now, so it is in line with the other isa values? I.e. the "prefix" part of the "future" isa. Or "futx"? Short names are important, keep it to 4 chars? (The isa values are named after the processor that first implements them, not the ISA versions, because 2.07 is much easier to get wrong than p8 is). Segher
