[PATCH] D36788: The issues with X86 prefixes

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

Please add a 32-bit mode test as well.



================
Comment at: lib/Target/X86/Disassembler/X86DisassemblerDecoder.cpp:993
     }
+  } else if (insn->mode == MODE_64BIT && !insn->mandatory_prefix) {
+    // If we don't have mandatory prefix we should use "standard" prefixes here
----------------
Is this 64-bit mode checks till needed?


https://reviews.llvm.org/D36788