[PATCH] D108289: [NFC][LoopIdiom] Let processLoopStoreOfLoopLoad take StoreSize as SCEV instead of unsigned

Yueh-Ting Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 18 04:15:53 PDT 2021


eopXD added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:861
   // that every byte is touched in the loop.
-  const SCEVConstant *StoreStride =
+  const SCEVConstant *ConstStoreStride =
       dyn_cast<SCEVConstant>(StoreEv->getOperand(1));
----------------
Renamed here for better readability. Also, future patch will deal with both constant / non-constant stride, so `StoreStride`, `LoadStride` will be in type `SCEV`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108289



More information about the llvm-commits mailing list