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.


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.

Wouldn't it work to do two entries, the POWER6 variant first? The table is searched in order, there are more things that depend on that already. Or would the assembler have issues with this?

I don't think it is worth the bother.

Fair enough. I think the whole "which insn set and insn set extensions are enabled" thing needs an overhaul, I just don't feel up to it :-)


Segher



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