[llvm-commits] [llvm] r152817 - in /llvm/trunk: include/llvm/MC/MCInstrInfo.h utils/TableGen/AsmMatcherEmitter.cpp utils/TableGen/AsmWriterEmitter.cpp utils/TableGen/InstrInfoEmitter.cpp utils/TableGen/StringToOffsetTable.h

[Jakob Stoklund Olesen]

On Mar 15, 2012, at 2:08 PM, Chris Lattner <clattner at apple.com> wrote:

> 
> On Mar 15, 2012, at 11:05 AM, Jakob Stoklund Olesen wrote:
> 
>> Author: stoklund
>> Date: Thu Mar 15 13:05:57 2012
>> New Revision: 152817
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=152817&view=rev
>> Log:
>> Revert r152202: "Use uint16_t to store InstrNameIndices in MCInstrInfo."
>> 
>> We cannot limit the concatenated instruction names to 64K.  ARM is
>> already at 32K, and it is easy to imagine a target with more
>> instructions.
> 
> Ok, but this also deconstifies OperandMatchEntry::MnemonicTable.  That shouldn't depend on how many names there are.

Ugh, missed that. I'll put it back.

/jakob