[PATCH] D104595: [NFC] [LoopIdiom] Let processLoopStridedStore take StoreSize as SCEV instead of unsigned

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 4 12:15:32 PDT 2021


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

Please make sure clang-format is good for the new lines you changed.
Otherwise, LGTM.



================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:941
   return processLoopStridedStore(
-      Pointer, (unsigned)SizeInBytes, MaybeAlign(MSI->getDestAlignment()),
-      SplatValue, MSI, MSIs, Ev, BECount, NegStride, /*IsLoopMemset=*/true);
+      Pointer, SE->getSCEV(MSI->getLength()), MaybeAlign(MSI->getDestAlignment()), SplatValue,
+      MSI, MSIs, Ev, BECount, NegStride, /*IsLoopMemset=*/true);
----------------
probably need to fix clang-format here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104595



More information about the llvm-commits mailing list