[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
Sat Jun 10 22:35:22 PDT 2023
vedant-amd added a comment.
In D152278#4408118 <https://reviews.llvm.org/D152278#4408118>, @vedant-amd wrote:
> 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.
>
> Hey @nikic, I did refactor the change, please take a look, all the SCEV test cases do pass with it. It look correct functionally.
ping ! could you please give a +1 ? @nikic @efriedma
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