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

[Sjoerd Meijer via llvm-commits]

SjoerdMeijer updated this revision to Diff 66330.
SjoerdMeijer added a comment.

It's funny but I haven't been able to create an extra test case that still does the reordering. The issue in creating that test case is, i) when I manually reorder blocks in the .ll files to create a layout predecessos that cannot fallthrough, then somehow the blocks still end up in the original order again and there is a fallthrough, and ii) even for conditional branches to the loop the canFallThrough function returns true, which again prevents me from creating a case that does not fallthrough.


https://reviews.llvm.org/D22521

Files:
  lib/CodeGen/MachineBlockPlacement.cpp
  test/CodeGen/X86/loop-blocks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22521.66330.patch
Type: text/x-patch
Size: 3634 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160801/86535f66/attachment.bin>