[PATCH] D99249: [PassManager] Run additional LICM before LoopRotate

[Nikita Popov via Phabricator via llvm-commits]

nikic added a reviewer: asbirlea.
nikic added a comment.

One thing I can throw out now is that currently, we use split LPMs for LoopRotate and LICM, because historically it was too expensive to preserve MSSA in LoopRotate. After your change MSSA needs to be preserved anyway, and we are in the rather silly situation of LICM, LoopRotate and LICM again each using a separate LPM. **If** we want to go down this route, we'd probably want to land something like D74640 <https://reviews.llvm.org/D74640> first in order to merge these LPMs again.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99249