[PATCH] D74640: [LoopRotate] Add explicit flag to require MSSA.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

Rebased after the fix to remove DT recomputation in MSSA. I'm still seeing a small regression on one of the tests in the original PRs (0.6% in instructions on mogrify.i), but it's nowhere near the regression that motivated the pipeline split.
I'm not sure if it's worth pushing this forward if the only motivation is re-merging the pipeline in the LPM, unless it's entirely performance neutral.

@nikic: Would you mind checking the compile-time impact for this patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74640