[PATCH] D60935: [IndVarSimplify] Fixup nowrap flags during LFTR when moving to post-inc (PR31181)
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 31 10:19:24 PDT 2019
reames accepted this revision.
reames added a comment.
Nikita and I talked offline. We decided that both the dead IV handling and gep inbounds cases raised previously are arguably distinct bugs in this code. The current patch does appear to fix one real issue (lftr post-incr flags), and that getting that functional fix in is valuable in the near term. Given that, I'm approving the review as it stands, and we're going to continue the discussion of the other possible issues in email, and then separate reviews for each.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60935/new/
https://reviews.llvm.org/D60935
More information about the llvm-commits
mailing list