[llvm-commits] [llvm] r140954 - in /llvm/trunk: lib/Target/X86/Disassembler/X86DisassemblerDecoderCommon.h test/MC/Disassembler/X86/simple-tests.txt test/MC/Disassembler/X86/x86-32.txt utils/TableGen/X86DisassemblerTables.cpp utils/TableGen/X86RecognizableInstr.cpp

[Bill Wendling]

On Oct 1, 2011, at 12:54 PM, Craig Topper wrote:

> Author: ctopper
> Date: Sat Oct  1 14:54:56 2011
> New Revision: 140954
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=140954&view=rev
> Log:
> Fix disassembler handling of CRC32 which is an odd instruction that uses 0xf2 as an opcode extension and allows the opsize prefix. This necessitated adding IC_XD_OPSIZE and IC_64BIT_XD_OPSIZE contexts. Unfortunately, this increases the size of the disassembler tables. Fixes PR10702.
> 
> Modified: llvm/trunk/utils/TableGen/X86DisassemblerTables.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/X86DisassemblerTables.cpp?rev=140954&r1=140953&r2=140954&view=diff
> ==============================================================================
> --- llvm/trunk/utils/TableGen/X86DisassemblerTables.cpp (original)
> +++ llvm/trunk/utils/TableGen/X86DisassemblerTables.cpp Sat Oct  1 14:54:56 2011
> @@ -51,8 +51,11 @@
>     return inheritsFrom(child, IC_64BIT_OPSIZE);
>   case IC_XD:
>     return inheritsFrom(child, IC_64BIT_XD);
> +           inheritsFrom(child, IC_XD_OPSIZE);

This looks odd...

-bw