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

[Kohei Asano via Phabricator via llvm-commits]

khei4 added a comment.

Although I can't reproduce this locally, I think this is suspicious.



================
Comment at: llvm/lib/Transforms/Scalar/MemCpyOptimizer.cpp:1632-1633
+  // seem worthwhile right now.
+  for (Instruction *I : NoAliasInstrs)
+    I->setMetadata(LLVMContext::MD_noalias, nullptr);
+
----------------
Hmm, it seems like setMetadata is called deleted instruction. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153453