[PATCH] D104180: [NFC] [LICM] Create LoopNest Invariant Code Motion (LNICM) pass

[Whitney Tsang via Phabricator via llvm-commits]

Whitney added a comment.

In D104180#2846602 <https://reviews.llvm.org/D104180#2846602>, @uint256_t wrote:

> We're thinking of replacing LICM in LPM1 with LNICM and adding LPM3 containing LICM after LPM2. 
> In long term, LPM3 should be removed though.

When you say "In long term, LPM3 should be removed though.", do you mean when LoopInterchange can preserve MSSA, then it could be move to LPM1, so LICM can be move to LPM2?
I think LICM is always needed in long term, correct?


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

https://reviews.llvm.org/D104180