[PATCH] D70030: [MachineBlockPlacement] Fix UnscheduledPredecessors counter to reflect tail duplication.

[Guozhi Wei via Phabricator via llvm-commits]

Carrot added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineBlockPlacement.cpp:2976
+      continue;
+    if (&BBChain != BlockToChain[Pred] && BBChain.UnscheduledPredecessors != 0)
+      --BBChain.UnscheduledPredecessors;
----------------
Here if BBChain.UnscheduledPredecessors == 0, then there is something wrong.


================
Comment at: llvm/lib/CodeGen/MachineBlockPlacement.cpp:2979
+
+    if (Pred == LPred || PredChain == &Chain)
       continue;
----------------
This condition also applies to your code, so the original if statement should not be changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70030