[clang] [llvm] [LPM][LegacyPM] Reenable LCSSA Verification (PR #191667)

Aiden Grossman via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 19 12:00:26 PDT 2026


================
@@ -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
----------------
boomanaiden154 wrote:

Good point. Done.

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


More information about the cfe-commits mailing list