[PATCH] D158716: [RFC][LV] VPlan-based cost model

[Wang Pengcheng via Phabricator via llvm-commits]

wangpc added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8700
+        C.first = VPCM.expectedCost(CostVF);
+        C.second = true;
+      } else
----------------
Is it always true even for scalar VF? Why can't we have the same signature as `CM.expectedCost`?


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanCostModel.h:30
+
+  /// Return cost of the VPlan for a given \p RVL
+  InstructionCost expectedCost(ElementCount VF);
----------------
`RVL`->`VF`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158716