[PATCH] D90563: [VPlan] Make VPWidenMemoryInstructionRecipe a VPDef.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1253
+                                       public VPUser,
+                                       public VPValue {
   void setMask(VPValue *Mask) {
----------------
gilr wrote:
> With this recipe being a VPDef we can drop the VPValue inheritance and create a VPValue (only) for loads, right?
That's a good point, we should be able to model this better now. Updated the patch


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90563