ImmMask should be 3 for a two-bit field; Compact X86II
authorEvan Cheng <evan.cheng@apple.com>
Thu, 18 May 2006 06:27:15 +0000 (06:27 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Thu, 18 May 2006 06:27:15 +0000 (06:27 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@28381 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86.td
lib/Target/X86/X86InstrInfo.h

index e310f2676c5660b6d72bcef554d57d6f56a0cbc3..e15512db23a0d63d8569d9d16d678d2e97ad548f 100644 (file)
@@ -114,7 +114,7 @@ def X86InstrInfo : InstrInfo {
                        7,
                        11,
                        13,
-                       17];
+                       16];
 }
 
 // The X86 target supports two different syntaxes for emitting machine code.
index 654a4334cfae918de1e7dd61d61cf820647f6c87..d6dfae14e6b752efff55caba112845d7a9e9edc1 100644 (file)
@@ -120,7 +120,7 @@ namespace X86II {
     // This two-bit field describes the size of an immediate operand.  Zero is
     // unused so that we can tell if we forgot to set a value.
     ImmShift = 11,
-    ImmMask  = 7 << ImmShift,
+    ImmMask  = 3 << ImmShift,
     Imm8     = 1 << ImmShift,
     Imm16    = 2 << ImmShift,
     Imm32    = 3 << ImmShift,
@@ -161,8 +161,7 @@ namespace X86II {
     // SpecialFP - Special instruction forms.  Dispatch by opcode explicitly.
     SpecialFP  = 7 << FPTypeShift,
 
-    // Bit 15 is unused.
-    OpcodeShift   = 17,
+    OpcodeShift   = 16,
     OpcodeMask    = 0xFF << OpcodeShift,
     // Bits 25 -> 31 are unused
   };