[PATCH] D149435: [LCSSA] Don't invalidate SCEV

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 05:27:48 PDT 2023


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

> However, nowadays SCEV itself no longer maintains LCSSA form. The SCEV of on LCSSA phi node will be the same as the SCEV of its argumen

Agreed, the phis we insert here should all be looked-through by SCEV. LGTM, thanks!


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

https://reviews.llvm.org/D149435



More information about the llvm-commits mailing list