[PATCH] D120827: [VPlan] Helper to check if a recipe only uses scalar values of op.

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1644
+
+  /// Returns true if the recipe only uses scalars of operand \p Op.
+  bool onlyScalarsUsed(const VPValue *Op) const override {
----------------
VPPredInstPHIRecipe has only PredInst as an operand, which is surely scalar.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1779
+
+  /// Returns true if the recipe only uses scalars of operand \p Op.
+  bool onlyScalarsUsed(const VPValue *Op) const override {
----------------
Does VPExpandSCEVRecipe use only first lane?


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1927
+
+  /// Returns true if the recipe only uses scalars of operand \p Op.
+  bool onlyScalarsUsed(const VPValue *Op) const override {
----------------
VPScalarIVStepsRecipe uses only the first lane of its three operands?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120827