[PATCH] D70157: Align branches within 32-Byte boundary

Kan Shengchen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 16 06:04:30 PST 2019


skan updated this revision to Diff 234041.
skan added a comment.

1. rename `getSegmentPrefixSize(const MCInst &MI)` to `countSegmentPrefix(const MCInst &MI)`.
2. remove unnecessary data member of X86AsmBackend `std::vector<MCMachineDependentFragment *> PendingAlignmentFragments`.
3. Make the message of `assert` in `MCMachineDependentFragment` more clear.


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

https://reviews.llvm.org/D70157

Files:
  clang/include/clang/Driver/Options.td
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/test/Driver/intel-align-branch.c
  clang/test/Driver/intel-align-branch.s
  llvm/include/llvm/MC/MCAsmBackend.h
  llvm/include/llvm/MC/MCAssembler.h
  llvm/include/llvm/MC/MCFragment.h
  llvm/include/llvm/MC/MCObjectStreamer.h
  llvm/lib/MC/MCAssembler.cpp
  llvm/lib/MC/MCFragment.cpp
  llvm/lib/MC/MCObjectStreamer.cpp
  llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
  llvm/lib/Target/X86/X86InstrInfo.td
  llvm/test/MC/X86/align-branch-32-1a.s
  llvm/test/MC/X86/align-branch-32-2a.s
  llvm/test/MC/X86/align-branch-32-3a.s
  llvm/test/MC/X86/align-branch-32-4a.s
  llvm/test/MC/X86/align-branch-64-1a.s
  llvm/test/MC/X86/align-branch-64-1b.s
  llvm/test/MC/X86/align-branch-64-1c.s
  llvm/test/MC/X86/align-branch-64-1d.s
  llvm/test/MC/X86/align-branch-64-2a.s
  llvm/test/MC/X86/align-branch-64-2b.s
  llvm/test/MC/X86/align-branch-64-2c.s
  llvm/test/MC/X86/align-branch-64-3a.s
  llvm/test/MC/X86/align-branch-64-4a.s
  llvm/test/MC/X86/align-branch-64-5a.s
  llvm/test/MC/X86/align-branch-64-6a.s
  llvm/test/MC/X86/align-branch-64-7a.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70157.234041.patch
Type: text/x-patch
Size: 118121 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191216/c917be4e/attachment-0001.bin>


More information about the cfe-commits mailing list