Don't mix declarations and code.
authorBenjamin Kramer <benny.kra@googlemail.com>
Sat, 11 Feb 2012 16:01:02 +0000 (16:01 +0000)
committerBenjamin Kramer <benny.kra@googlemail.com>
Sat, 11 Feb 2012 16:01:02 +0000 (16:01 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@150305 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/Disassembler/X86DisassemblerDecoder.c
lib/Target/X86/Disassembler/X86DisassemblerDecoder.h

index c15709d33dae3a13e51227b9cba4e2a52e8bc0f7..72ae8552027e0424c4b707fd111b72deaff69584 100644 (file)
@@ -826,7 +826,7 @@ static int getID(struct InternalInstruction* insn, void *miiArg) {
     
     const struct InstructionSpecifier *spec;
     uint16_t instructionIDWithOpsize;
-    const struct InstructionSpecifier *specWithOpsize;
+    const char *specName, *specWithOpSizeName;
     
     spec = specifierForUID(instructionID);
     
@@ -843,15 +843,13 @@ static int getID(struct InternalInstruction* insn, void *miiArg) {
       return 0;
     }
     
-    specWithOpsize = specifierForUID(instructionIDWithOpsize);
-
-    const char *specName = x86DisassemblerGetInstrName(instructionID, miiArg);
-    const char *specWithOpSizeSizeName =
+    specName = x86DisassemblerGetInstrName(instructionID, miiArg);
+    specWithOpSizeName =
       x86DisassemblerGetInstrName(instructionIDWithOpsize, miiArg);
 
-    if (is16BitEquvalent(specName, specWithOpSizeSizeName)) {
+    if (is16BitEquvalent(specName, specWithOpSizeName)) {
       insn->instructionID = instructionIDWithOpsize;
-      insn->spec = specWithOpsize;
+      insn->spec = specifierForUID(instructionIDWithOpsize);
     } else {
       insn->instructionID = instructionID;
       insn->spec = spec;
index c2a40ae0d3c8074e9729f7e4aa7a99ffc8158334..7e2667614fd08321843dd17af143e4027ed62c3d 100644 (file)
@@ -20,7 +20,7 @@
 extern "C" {
 #endif
   
-#define INSTRUCTION_SPECIFIER_FIELDS  \
+#define INSTRUCTION_SPECIFIER_FIELDS
 
 #define INSTRUCTION_IDS     \
   unsigned instructionIDs;