[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