[PATCH] D152278: [SCEV] Compute SCEV for ashr(add(shl(x, n), c), m) instr triplet
Vedant Paranjape via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 9 01:37:38 PDT 2023
vedant-amd added a comment.
In D152278#4407841 <https://reviews.llvm.org/D152278#4407841>, @nikic wrote:
> I assume the motivation for this is the following InstCombine transform? https://llvm.godbolt.org/z/zd63TWKbW I don't think we can change that canonicalization, so undoing it in SCEV is fine.
Yup, that's correct. This canonicalization is used several times in LSR, so having it's SCEV might be beneficial.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152278/new/
https://reviews.llvm.org/D152278
More information about the llvm-commits
mailing list