[PATCH] D151698: [SCEV] Compute AddRec range computations using different type BECount

Joshua Cao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 02:16:25 PDT 2023


caojoshua created this revision.
Herald added a subscriber: hiraditya.
Herald added a project: All.
caojoshua added reviewers: nikic, fhahn, mkazantsev.
Herald added a subscriber: StephenFan.
caojoshua added a reviewer: reames.
caojoshua updated this revision to Diff 526552.
caojoshua edited the summary of this revision.
caojoshua added a comment.
caojoshua published this revision for review.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Clean up code a bit


caojoshua added a comment.

I've no idea if the Dependence Analysis test change is correct. Would appreciate if reviews can look at it.


Before this patch, we can only use the MaxBECount for an AddRec's range
computation if the MaxBECount has <= bit width of the AddRec. This patch
reasons that if a MaxBECount has > bit width, and is <= the max value of
AddRec's bit width, we can still use the MaxBECount.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D151698

Files:
  llvm/lib/Analysis/ScalarEvolution.cpp
  llvm/test/Analysis/DependenceAnalysis/NonCanonicalizedSubscript.ll
  llvm/test/Analysis/ScalarEvolution/increasing-or-decreasing-iv.ll
  llvm/test/Analysis/ScalarEvolution/limit-depth.ll
  llvm/test/Analysis/ScalarEvolution/nsw.ll
  llvm/test/Analysis/ScalarEvolution/trip-count-negative-stride.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151698.526552.patch
Type: text/x-patch
Size: 11580 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230530/b8ba1d4b/attachment.bin>


More information about the llvm-commits mailing list