[PATCH] D135137: [AggressiveInstCombine] Load merge the reverse load pattern of consecutive loads.

Thu Nov 3 05:17:46 PDT 2022

bipmis added a comment.

In D135137#3904719 <https://reviews.llvm.org/D135137#3904719>, @dmgreen wrote:

> Hello, as mentioned above there is a fix in https://reviews.llvm.org/D137201. We found another case that was incorrect though, so were looking into fixing that too. If it does take too long, perhaps a revert is in order in the meantime.
>
> Did you get a chance to verify the fix worked for your case? Thanks.

I think as a quick fix we can switch off load merge when we encounter stores and eventually come with a proper fix with additional test cases. I will do the patch on the same so that this does not block us.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135137