[PATCH] D110922: [LoopPeel] Peel loops with deoptimizing exits

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 5 01:28:02 PDT 2021


mkazantsev added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopPeel.cpp:757
+      for (auto Exit : ExitEdges)
+        DT->insertEdge(cast<BasicBlock>(LVMap[Exit.first]), Exit.second);
 #ifdef EXPENSIVE_CHECKS
----------------
Instead of this, please fill a SmallVector with DomTreeUpdates and then `DT->applyUpdates` once. It's a more canonical way of doing this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110922



More information about the llvm-commits mailing list