[PATCH] D22521: [MBP] do not reorder and move up loop latch block

[David Li via llvm-commits]

davidxl added inline comments.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:928
@@ +927,3 @@
+  // code size, so we don't do this reordering when optimising for size.
+  if (L.getHeader()->getParent()->getFunction()->optForSize())
+    return L.getHeader();
----------------
There is one case where the existing header is not the cfg successor of its layout predecessor. In this case, rotating loop does not actually add additional size cost.


https://reviews.llvm.org/D22521