[PATCH] D130233: [LoopLoadElim] Add stores with matching sizes as load-store candidates

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 07:37:06 PDT 2022


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

LG



================
Comment at: llvm/lib/Transforms/Scalar/LoopLoadElimination.cpp:101
     Type *LoadType = getLoadStoreType(Load);
+    Type *StoreType = getLoadStoreType(Store);
+    auto &DL = Load->getParent()->getModule()->getDataLayout();
----------------
Needs a `(void)StoreType`, or move the call into the assert.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130233



More information about the llvm-commits mailing list