[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