[PATCH] D75203: Relax existing instructions to reduce the number of nops needed for alignment purposes

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 11:35:29 PST 2020


reames updated this revision to Diff 247044.
reames added a comment.

Restructure code triggered by discussion on D75268 <https://reviews.llvm.org/D75268>.  If I simply admit that padding instructions is inherently target specific, we have a nice clean API for any form of target specific padding.  As a separate patch, I'll add prefix padding to this API for x86.  It also appears that Hexagon has a bundle nop padding (currently via finishLayout) which can be refactored into this API as well.


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

https://reviews.llvm.org/D75203

Files:
  llvm/include/llvm/MC/MCAsmBackend.h
  llvm/include/llvm/MC/MCAssembler.h
  llvm/lib/MC/MCAssembler.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
  llvm/test/MC/X86/align-branch-64.s
  llvm/test/MC/X86/align-via-relaxation.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75203.247044.patch
Type: text/x-patch
Size: 12738 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200227/b5db83bc/attachment.bin>


More information about the llvm-commits mailing list