[PATCH] D152278: [SCEV] Compute SCEV for ashr(add(shl(x, n), c), m) instr triplet

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