[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