[PATCH] D109457: [SCEV] Use constant range of RHS to prove NUW on narrow IV in trip count logic

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 16 12:49:41 PDT 2021


reames updated this revision to Diff 373037.
reames added a comment.

Rebase over landed tests.

One question for reviewers - We don't currently get any value in handling non-constant strides because (due to limitations in flag inference), we basically never conclude the that step is non-zero from data flow.  (We might from control flow, but that doesn't influence constant ranges.)  Should I keep the complexity?  Or just drop it to constant non-zero step and be done?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109457/new/

https://reviews.llvm.org/D109457

Files:
  llvm/lib/Analysis/ScalarEvolution.cpp
  llvm/test/Analysis/ScalarEvolution/trip-count-implied-addrec.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109457.373037.patch
Type: text/x-patch
Size: 6100 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210916/c5c5d3a7/attachment.bin>


More information about the llvm-commits mailing list