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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 21 13:22:24 PST 2019


davidxl added inline comments.


================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:2008
+        BlockChain *SuccChain = BlockToChain[Succ];
+        if ((!SuccChain || Succ == *SuccChain->begin()) && SuccProb > TopProb) {
+          TopOK = false;
----------------
What is the purpose of the check 'Succ == *SuccChain->begin() ?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58393/new/

https://reviews.llvm.org/D58393





More information about the llvm-commits mailing list