[PATCH] D101289: [SimplifyCFG] Use applyUpdatesPermissive in HoistThenElseCodeToIf.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 26 05:57:22 PDT 2021


lebedev.ri added a comment.

How about this then?



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:1620-1622
   if (DTU)
     for (BasicBlock *Succ : successors(BI))
       Updates.push_back({DominatorTree::Delete, BIParent, Succ});
----------------



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:1626
   if (DTU)
-    DTU->applyUpdates(Updates);
+    DTU->applyUpdatesPermissive(Updates);
   return Changed;
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101289



More information about the llvm-commits mailing list