Now that SCEV has a dedicated vscale node type, we should also map vscale intrinsics to it. To make sure this does not regress ranges (which were KnownBits based previously), add support for vscale to getRangeRef() as well.
Depends on D146217.
Paths
| Differential D146226
[SCEV] Recognize vscale intrinsics ClosedPublic Authored by nikic on Mar 16 2023, 6:44 AM.
Details Summary Now that SCEV has a dedicated vscale node type, we should also map vscale intrinsics to it. To make sure this does not regress ranges (which were KnownBits based previously), add support for vscale to getRangeRef() as well. Depends on D146217.
Diff Detail
Event TimelineComment Actions LGTM w/minor comment
This revision is now accepted and ready to land.Mar 16 2023, 7:44 AM Closed by commit rGa5242483e40c: [SCEV] Recognize vscale intrinsics (authored by nikic). · Explain WhyMar 17 2023, 2:09 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 506010 llvm/include/llvm/Analysis/ValueTracking.h
llvm/lib/Analysis/ScalarEvolution.cpp
llvm/lib/Analysis/ValueTracking.cpp
llvm/test/Analysis/ScalarEvolution/scalable-vector.ll
|
Can you rephrase this comment? This doesn't just bound the vscale intrinsic, it also bounds the vscale type param.