[PATCH] D153453: [MemCpyOpt] implement single BB stack-move optimization which unify the static unescaped allocas

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D153453#4524994 <https://reviews.llvm.org/D153453#4524994>, @khei4 wrote:

> @nikic Thanks! I added a test <https://reviews.llvm.org/rGd878916c213c9184e0cd5032a246580d8c3b6bea> and prevented insertion of lifetime intrinsic to NoAliasInstrs on CaptureTracking lambda function. Something odd is I can't reproduce reported errors on that test, which have noalias metadata on lifetime intrinsics.

Do you get memory errors when running under `valgrind`? Use after free doesn't always result in a crash.


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

https://reviews.llvm.org/D153453