[PATCH] D147378: [LV] Replace symbolic stride with const in LV

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 17:41:45 PDT 2023


reames added a comment.

This seems like it's duplicating information already available in PredicatedScalarEvolution.

You can see my D147750 <https://reviews.llvm.org/D147750> for one example where PSE solves a related problem.  I suspect we should probably be phrasing this one in terms of constant folding and instsimplify where the constants might come from PSE.  Maybe this doesn't work out, but I'd suggest trying that before adding additional complexity here.


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

https://reviews.llvm.org/D147378



More information about the llvm-commits mailing list