[PATCH] [MC] Make bundle alignment mode setting idempotent and support nested bundles

Derek Schuff dschuff at google.com
Wed Oct 15 09:17:37 PDT 2014


Hi jvoung, eliben,

Currently an error is thrown if bundle alignment mode is set more than once
per module (either via the API or the .bundle_align_mode directive). This
change allows setting it multiple times as long as the alignment doesn't
change.

Also nested bundle_lock groups are currently not allowed. This change allows
them, with the effect that the group stays open until all nests are exited,
and if any of the bundle_lock directives has the align_to_end flag, the
group becomes align_to_end.

These changes make the bundle aligment simpler to use in the compiler, and
also better match the corresponding support in GNU as.

http://reviews.llvm.org/D5801

Files:
  include/llvm/MC/MCAssembler.h
  lib/MC/MCAssembler.cpp
  lib/MC/MCELFStreamer.cpp
  test/MC/X86/AlignedBundling/nesting.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5801.14947.patch
Type: text/x-patch
Size: 6055 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141015/f03aa12f/attachment.bin>


More information about the llvm-commits mailing list