[PATCH] D43256: [MBP] Move a latch block with conditional exit and multi predecessors to top of loop

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: test/CodeGen/X86/move_latch_to_loop_top.ll:38
+;CHECK-LABEL: test2:
+;CHECK:       %latch
+;CHECK:       %header
----------------
This is not the optimal rotation for the loop. The optimal rotation should be

true
latch
header
false


in which case there is one more fall through from true to latch.

I think the enhancement should shoot for getting the optimal layout.


https://reviews.llvm.org/D43256