[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