[PATCH] D34566: [loop idiom Recognition] support memcpy for multiple consecutive loads and stores

DIVYA SHANMUGHAN via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 13:10:41 PDT 2017


DIVYA marked an inline comment as done.
DIVYA added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopIdiomRecognize.cpp:737
+    } else {
+        if (processLoopStoreOfLoopLoad(StorePtr, StoreSize, HeadStore,
+                                     StoreLoadInst, AdjacentStores, StoreEv,
----------------
haicheng wrote:
> I think processLoopStoreOfLoopLoad() can refactor with processLoopStridedStore(), then you don't need if...else... here.
processLoopStoreOfLoopLoad() function was already present,so I haven't refactored it with  processLoopStridedStore()  in this patch.I can do that in the next patch


https://reviews.llvm.org/D34566





More information about the llvm-commits mailing list