[PATCH] D26169: [Reassociate] Skip reassociation for IV if its def instruction contains NSW/NUW flags to enable better IndVar and LSR

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 8 18:56:36 PST 2016


sanjoy requested changes to this revision.
sanjoy added a comment.
This revision now requires changes to proceed.

This can potentially be a compile time issue.  I think the SCEV dependency itself is fine, since SCEV will do things lazily, but computing the LoopInfo can be a problem.

Can this be solved by some changes to our pass order?  I.e. first run indvars and then re-associate?


Repository:
  rL LLVM

https://reviews.llvm.org/D26169





More information about the llvm-commits mailing list