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

[Mandeep Singh Grang via Phabricator via llvm-commits]

mgrang added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopIdiomRecognize.cpp:901
 
-/// If the stored value is a strided load in the same loop with the same stride
-/// this may be transformable into a memcpy.  This kicks in for stuff like
+////// If there are one or multiple stored values , which are strided loads
+//in the same loop with the same stride , then this amy be transformed into
----------------
Extra / in comments. Also check comment spacing.


================
Comment at: test/Transforms/LoopIdiom/memcpy_structPattern.ll:1
+; RUN: opt -globals-aa -loop-idiom < %s -S | FileCheck %s
+
----------------
Unit test names follow all lowercase naming convention: memcpy_struct_pattern.ll


https://reviews.llvm.org/D34566