[llvm] [VPlan] First step towards VPlan cost modeling. (PR #67934)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Fri May 10 07:43:35 PDT 2024
================
@@ -340,6 +340,9 @@ class LoopVectorizationPlanner {
/// A builder used to construct the current plan.
VPBuilder Builder;
+ /// Computes the cost of \p Plan for vectorization factor \p VF.
----------------
fhahn wrote:
The current implementation requires access to the legacy cost model and I kept the various computeCost implementations that still require it separate from the VPlan-only infrastructure, to avoid leaking/polluting the VPlan-only cost implementations. I'd propose moving it to `VPlan::computeCost` once the legacy cost model isn't needed any longer.
Added a comment + TODO
https://github.com/llvm/llvm-project/pull/67934
More information about the llvm-commits
mailing list