[PATCH] D116930: [DAGCombine] Fold SRA of a load into a narrower sign-extending load

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 06:06:28 PST 2022


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

In D116930#3265911 <https://reviews.llvm.org/D116930#3265911>, @RKSimon wrote:

> Does this need to be rebased after rG46cacdbb21c2 <https://reviews.llvm.org/rG46cacdbb21c221a4304c489cb4a1abbc51967bb1>?
>
> Looks a lot nicer now - @spatel any more comments?

LGTM. This part is relatively straightforward now - assuming we got the preceding steps right (might want to give that a little time to bake in the wild).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116930/new/

https://reviews.llvm.org/D116930



More information about the llvm-commits mailing list