[PATCH] D113028: [LoopPeel] Add EXPENSIVE_CHECKS ifdef guard around domtree verify call

Stephen Long via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 2 09:39:51 PDT 2021


steplong created this revision.
Herald added a subscriber: hiraditya.
steplong requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D113028

Files:
  llvm/lib/Transforms/Utils/LoopPeel.cpp


Index: llvm/lib/Transforms/Utils/LoopPeel.cpp
===================================================================
--- llvm/lib/Transforms/Utils/LoopPeel.cpp
+++ llvm/lib/Transforms/Utils/LoopPeel.cpp
@@ -892,8 +892,10 @@
   // We modified the loop, update SE.
   SE->forgetTopmostLoop(L);
 
+#ifdef EXPENSIVE_CHECKS
   // Finally DomtTree must be correct.
   assert(DT->verify(DominatorTree::VerificationLevel::Fast));
+#endif
 
   // FIXME: Incrementally update loop-simplify
   simplifyLoop(L, DT, LI, SE, AC, nullptr, PreserveLCSSA);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113028.384142.patch
Type: text/x-patch
Size: 537 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211102/cfd99442/attachment.bin>


More information about the llvm-commits mailing list