[PATCH] D15894: Implement load to store => memcpy in MemCpyOpt for aggregates

[Mehdi AMINI via llvm-commits]

joker.eph added a subscriber: joker.eph.

================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:528
@@ +527,3 @@
+          }
+        }
+
----------------
one line comment this loop and `AI`

================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:533
@@ +532,3 @@
+          // source and dest might overlap.  We still want to eliminate the intermediate
+          // value, but we have to generate a memmove instead of memcpy.
+          bool UseMemMove = false;
----------------
not sure what you mean by " If the dest of the second might alias the source of the first", this sentence seems confusing to me. Is there more than `// If the load and the store alias, we have to use a memmove instead of a memcpy.` ?


http://reviews.llvm.org/D15894