[PATCH] D21087: [MemCpyOpt] Do not exchange llvm.lifetime.start and llvm.memcpy

[Eric Christopher via llvm-commits]

echristo added inline comments.

================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:792
@@ +791,3 @@
+  // Lifetime marks shouldn't be operated on.
+  if (Function *F = C->getCalledFunction())
+    if (F->isIntrinsic() && F->getIntrinsicID() == Intrinsic::lifetime_start)
----------------
We might be able to move this check to line 1184 before we call the function... or maybe hoist the conditions a bit and handle it for processStore as well?

Thoughts?


http://reviews.llvm.org/D21087