[PATCH] D37262: The issues with X86 prefixes: step 2

[Andrew V. Tischenko via Phabricator via llvm-commits]

avt77 updated this revision to Diff 117520.
avt77 added a comment.

I re-implemented assembler in case of working with X86-prefixes. Now both X86-assembler and X86-disassembler use the new Flags field from MCInst. As result now it's possible to track several prefixes for one instr and now one prefix is not a separate instruction but only is the parameter of the one. I tried to keep the current tests unchanged where it's possible. And I did not extend/change any diagnostic related to prefixes: it should be done in the follow up patches.


https://reviews.llvm.org/D37262

Files:
  include/llvm/MC/MCInst.h
  lib/Target/X86/AsmParser/X86AsmParser.cpp
  lib/Target/X86/AsmParser/X86Operand.h
  lib/Target/X86/Disassembler/X86Disassembler.cpp
  lib/Target/X86/Disassembler/X86DisassemblerDecoder.cpp
  lib/Target/X86/Disassembler/X86DisassemblerDecoder.h
  lib/Target/X86/InstPrinter/X86ATTInstPrinter.cpp
  lib/Target/X86/InstPrinter/X86IntelInstPrinter.cpp
  lib/Target/X86/MCTargetDesc/X86BaseInfo.h
  lib/Target/X86/MCTargetDesc/X86MCCodeEmitter.cpp
  test/MC/Disassembler/X86/prefixes.txt
  test/MC/X86/intel-syntax-encoding.s
  test/MC/X86/x86-64.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37262.117520.patch
Type: text/x-patch
Size: 35295 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171003/bf65dde0/attachment.bin>