[llvm-branch-commits] [llvm] [LPM][LegacyPM] Reenable LCSSA Verification (PR #191667)

[Ramkumar Ramachandra via llvm-branch-commits]

================
@@ -239,12 +239,8 @@ bool LPPassManager::runOnFunction(Function &F) {
         // is that LPPassManager might run passes which do not require LCSSA
         // form (LoopPassPrinter for example). We should skip verification for
         // such passes.
-        // FIXME: Loop-sink currently break LCSSA. Fix it and reenable the
-        // verification!
-#if 0
         if (mustPreserveAnalysisID(LCSSAVerificationPass::ID))
           assert(CurrentLoop->isRecursivelyLCSSAForm(*DT, *LI));
-#endif
----------------
artagnon wrote:

Hm, shouldn't this be guarded under debug? DT is guarded under debug?

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