[PATCH] D84682: [VPlan] Use VPValue def for VPWidenSelectRecipe.

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:852
                       bool InvariantCond)
-      : VPRecipeBase(VPWidenSelectSC), Ingredient(I), User(Operands),
-        InvariantCond(InvariantCond) {}
+      : VPRecipeBase(VPWidenSelectSC), VPValue(VPValue::VPVWidenSelectSC, &I),
+        User(Operands), InvariantCond(InvariantCond) {}
----------------
VPRecipeBase::VPWidenSelectSC


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:869
+
+  SelectInst *getUnderlyingInstruction() {
+    return cast<SelectInst>(getUnderlyingValue());
----------------
Add a const version too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84682