[PATCH] D84680: [VPlan] Use VPValue def for VPMemoryInstructionRecipe.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:7821
+        if (!Member->getType()->isVoidTy()) {
+          NewVPV = new VPValue(Member);
+          Plan->getVPValue(Member)->replaceAllUsesWith(NewVPV);
----------------
vitalybuka wrote:
> this is a memory leak 
> http://lab.llvm.org:8014/#/builders/68/builds/1/steps/12/logs/stdio
Thanks! Recommitted with a fix and it seems like the staging msan bit is happy now


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84680