[PATCH] D58393: [MBP] Factor out function hasViableTopFallthrough and enhancement

[Guozhi Wei via Phabricator via llvm-commits]

Carrot created this revision.
Carrot added a reviewer: davidxl.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This patch factor out the function hasViableTopFallthrough from rotateLoop. It is also enhanced. Original code checks only if there is a block can be placed before current loop top. This patch also checks if the loop top is the most possible successor of its predecessor. The attached test case shows its effect.


Repository:
  rL LLVM

https://reviews.llvm.org/D58393

Files:
  lib/CodeGen/MachineBlockPlacement.cpp
  test/CodeGen/X86/bb_rotate.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58393.187409.patch
Type: text/x-patch
Size: 3824 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190219/99f4b069/attachment.bin>