[PATCH] D76398: [X86] Limit prefix padding w/target specific padding amount

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 10:52:23 PDT 2020


reames updated this revision to Diff 251413.
reames retitled this revision from "[X86] Enable prefix padding w/target specific padding amount" to "[X86] Limit prefix padding w/target specific padding amount".
reames edited the summary of this revision.
reames added a comment.

Restructure patch not to enable by default.  This allows the target specific logic to be tested in tree while avoiding the optimized assembler issue for now.

@skan - Will reply to your point separately via email.  I disagree and think we need a broader discussion on this point.


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

https://reviews.llvm.org/D76398

Files:
  llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
  llvm/test/MC/X86/prefix-padding-32.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76398.251413.patch
Type: text/x-patch
Size: 5919 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200319/10ef2128/attachment.bin>


More information about the llvm-commits mailing list