[PATCH] D38676: [LV] Model masking in VPlan, introducing VPInstructions

[Gil Rapaport via Phabricator via llvm-commits]

gilr marked 5 inline comments as done.
gilr added a comment.

Thanks, Amara!



================
Comment at: lib/Transforms/Vectorize/VPlan.h:736
+    for (auto &MapEntry : Value2VPValue)
+      delete MapEntry.second;
   }
----------------
aemerson wrote:
> Why delete the value from the map but leave the key?
These are Values from the existing IR code - VPlan doesn't own them, just maintains a VPValue to represent them at the VPlan-level Def-Use graph.


https://reviews.llvm.org/D38676