[PATCH] D38676: [LV] Model masking in VPlan, introducing VPInstructions
Gil Rapaport via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 10 06:31:57 PDT 2017
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
More information about the llvm-commits
mailing list