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

[David Stuttard via Phabricator via llvm-commits]

dstuttard added inline comments.


================
Comment at: llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp:816
+      IntegerType::get(Load1Ptr->getContext(), LOps.LoadSize), Load1Ptr,
+      LI1->getAlign(), LI1->isVolatile(), "");
+  NewLoad->takeName(LI1);
----------------
Kai wrote:
> FYI: The missing bitcast for the `Load1Ptr` argument means that this change only works with opaque pointers.
We had the same issue - I've just uploaded a patch. See D135249



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127392