This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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] Update mtfsf and mtfsfi instructions to support new optional operands.

On Thu, May 17, 2007 at 02:57:11PM +0200, Segher Boessenkool wrote:
> >-{ "mtfsf",   XFL(63,711,0), XFL_MASK,	COM,		{ FLM, FRB } 
> >},
> >-{ "mtfsf.",  XFL(63,711,1), XFL_MASK,	COM,		{ FLM, FRB } 
> >},
> >+{ "mtfsf",   XFL(63,711,0), XFL_MASK,	COM,		{ FLM, FRB, 
> >XFL_L, XFL_W } },
> >+{ "mtfsf.",  XFL(63,711,1), XFL_MASK,	COM,		{ FLM, FRB, 
> >XFL_L, XFL_W } },
> 
> Shouldn't the new forms be marked POWER6-only, not COM?

Yes, but then you'd have two entries for the same opcode and you'd
need to introduce a NOPOWER6 flag so that only one entry was used.
I don't think it is worth the bother.

-- 
Alan Modra
IBM OzLabs - Linux Technology Centre
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]