[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