[all-commits] [llvm/llvm-project] d34661: [LPM][LegacyPM] Reenable LCSSA Verification

Aiden Grossman via All-commits all-commits at lists.llvm.org
Sun Apr 19 12:02:46 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d34661b89c7978b28005811c557ca8aca1a23d0d
      https://github.com/llvm/llvm-project/commit/d34661b89c7978b28005811c557ca8aca1a23d0d
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2026-04-19 (Sun, 19 Apr 2026)

  Changed paths:
    M llvm/lib/Analysis/LoopPass.cpp

  Log Message:
  -----------
  [LPM][LegacyPM] Reenable LCSSA Verification

This was disabled about a decade ago due to issues with LoopSink.
LoopSink has since had its LegacyPM version removed and is now a
function pass due to not needing too much loop infrastructure. So we can
try enabling this again to prevent backsliding on important cases while
we work on switching to the NewPM which does enforce these things.

Eventually we will want to add assertions here for LoopStrengthReduce,
but given it does not correctly preserve LCSSA, postpone that for now.

Reviewers: arsenm, Meinersbur, nikic, fhahn

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



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list