[PATCH] D127392: [AggressiveInstCombine] Combine consecutive loads which are being merged to form a wider load.

Fri Jul 15 05:00:31 PDT 2022

spatel added a comment.

In D127392#3654507 <https://reviews.llvm.org/D127392#3654507>, @bipmis wrote:

> I should have mentioned. This being the base version have not enabled the same. Just targeting simple load scenarios in this patch. This was enabled in the the InstCombine patch.  The same will be enabled in the subsequent patches to handle memory access b/w loads.

I don't understand the comment. Does the posted patch miscompile the example with a store? If so, then the patch can't be committed in this form. 
If there are planned changes to the posted patch before it is ready for review, please note that in the patch status (or add "WIP" to the title).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127392