[PATCH] D19707: [MBP] Split block placement and block alignment. NFC.

Amaury SECHET via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 28 21:06:05 PDT 2016


deadalnix accepted this revision.
deadalnix added a reviewer: deadalnix.
deadalnix added a comment.
This revision is now accepted and ready to land.

LGTM. Please give it some time for other to comment before landing.


================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1206
@@ +1205,3 @@
+  // loop rotations done during this layout pass.
+  // FIXME: Use Function::optForSize().
+  if (F.getFunction()->hasFnAttribute(Attribute::OptimizeForSize))
----------------
I assume this is the next step.


Repository:
  rL LLVM

http://reviews.llvm.org/D19707





More information about the llvm-commits mailing list