[PATCH] D71315: [WIP] Draft assembler support for branch alignment

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 16:36:55 PST 2019


reames created this revision.
Herald added subscribers: llvm-commits, bollu, hiraditya, mcrosier.
Herald added a project: LLVM.
reames edited the summary of this revision.

Inspired by discussion on D71238 <https://reviews.llvm.org/D71238>.  Please direct all discussion there until direction is agreed on.  This isn't worthy of review until we settle on direction and syntax, it's just posted for context.

In particular, this patch is: a) using unsettled syntax which will likely change, b) hasn't merged changes in the alignment logic previously suggested, and c) is missing a whole bunch of tests.

The strawman syntax used in this patch is:

  .boundary_align 5
  testq %rax, %rax
  jnz baz
  .boundary_align_end

I picked this mostly for ease of implementation, nothing more.


https://reviews.llvm.org/D71315

Files:
  llvm/include/llvm/MC/MCAssembler.h
  llvm/include/llvm/MC/MCFragment.h
  llvm/include/llvm/MC/MCObjectStreamer.h
  llvm/include/llvm/MC/MCStreamer.h
  llvm/lib/MC/MCAsmStreamer.cpp
  llvm/lib/MC/MCAssembler.cpp
  llvm/lib/MC/MCFragment.cpp
  llvm/lib/MC/MCObjectStreamer.cpp
  llvm/lib/MC/MCParser/AsmParser.cpp
  llvm/lib/MC/MCStreamer.cpp
  llvm/test/MC/X86/align-branch-64.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71315.233224.patch
Type: text/x-patch
Size: 19339 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191211/4800593d/attachment.bin>


More information about the llvm-commits mailing list