[PATCH] D155406: [MemCpyOpt] implement multi BB stack-move optimization

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka accepted this revision.
vitalybuka added a comment.

Yes, like this.

LGTM,

If you can wait a few hours I try the patch on our internal code.



================
Comment at: llvm/lib/Transforms/Scalar/MemCpyOptimizer.cpp:1611
+  // exists.
   if (!LifetimeMarkers.empty()) {
     for (Instruction *I : LifetimeMarkers)
----------------
Please drop TODO: Reconstruct merged lifetime markers.


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

https://reviews.llvm.org/D155406