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

Sat Jul 22 18:27:17 PDT 2023

khei4 added a comment.

In D153453#4525016 <https://reviews.llvm.org/D153453#4525016>, @nikic wrote:

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

Ah, yes. `valgrind  --track-origins=yes` found the one for the added test! Thanks, it's beneficial!

  ==1060872== Command: ./build/bin/opt -passes=memcpyopt -S ./test.ll
  ==1060872== 
  ==1060872== Invalid read of size 8
  ==1060872==    at 0x271F568: llvm::Instruction::setMetadata(unsigned int, llvm::MDNode*) ...

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

https://reviews.llvm.org/D153453