[PATCH] D120592: [X86] Preserve redundant Address-Size override prefix

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 2 15:39:15 PST 2022


Amir updated this revision to Diff 412564.
Amir added a comment.
Herald added a subscriber: arphaman.

Moved RawFrmSrc, RawFrmDstSrc, RawFrmDst handling from X86MCCodeEmitter::emitPrefixImpl
to X86_MC::needsAddressSizeOverride. This consolidates all checks controlling
the emission of AddressSize override prefix in needsAddressSizeOverride.

The flipside is a prefix order change but it should be benign.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120592/new/

https://reviews.llvm.org/D120592

Files:
  llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.h
  llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86MCCodeEmitter.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.h
  llvm/test/MC/Disassembler/X86/addr32.s
  llvm/test/MC/X86/code16gcc.s
  llvm/test/MC/X86/index-operations.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120592.412564.patch
Type: text/x-patch
Size: 21734 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220302/f005dcf7/attachment.bin>


More information about the llvm-commits mailing list