[PATCH] D61934: [SCEV] Use wrap flags in InsertBinop
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 6 10:33:01 PDT 2019
reames added a comment.
Flags on SCEVs are valid for the current set of users that exist in the loop. If you add *new* users then those flags may no longer be valid. We recently ran into this in LFTR, and this patch seems likely to have the same risk. I have not audited the patch or its output to be sure, just throwing out a possibility.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61934/new/
https://reviews.llvm.org/D61934
More information about the llvm-commits
mailing list