[llvm] [CodeGen] Remove `applySplitCriticalEdges` in `MachineDominatorTree` (PR #97055)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 3 01:30:12 PDT 2024


================
@@ -391,9 +390,8 @@ bool MachineLICMBase::runOnMachineFunction(MachineFunction &MF) {
     else {
       // CSEMap is initialized for loop header when the first instruction is
       // being hoisted.
-      MachineDomTreeNode *N = DT->getNode(CurLoop->getHeader());
       FirstInLoop = true;
-      HoistOutOfLoop(N, CurLoop, CurPreheader);
----------------
paperchalice wrote:

`N` is no longer a node in `DT` when we split critical edges eagerly in failed tests before.

https://github.com/llvm/llvm-project/pull/97055


More information about the llvm-commits mailing list