[PATCH] D97667: [loop-idiom] Hoist loop memcpys to loop preheader

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 30 15:32:38 PDT 2021


lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:865
+  APInt LoadIntStride = LoadStride->getAPInt();
+  if (SizeInBytes != StoreIntStride && SizeInBytes != -StoreIntStride) {
+    ORE.emit([&]() {
----------------
I don't think this should handle the case of different directions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97667



More information about the llvm-commits mailing list