[PATCH] D15648: Add command line options to force function/loop alignments.
Chad Rosier via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 29 10:21:04 PST 2015
mcrosier accepted this revision.
mcrosier added a reviewer: mcrosier.
mcrosier added a comment.
This revision is now accepted and ready to land.
In http://reviews.llvm.org/D15648#315796, @gberry wrote:
> LGTM with one caveat: we should double-check that the MachineBlock emission handles the case where the padding instruction bytes could be executed (i.e. that it knows to emit nops instead of just zeros).
I didn't run into any issues across Spec200X. LGTM, per Geoff.
http://reviews.llvm.org/D15648
More information about the llvm-commits
mailing list