llvm / llvm/llvm-project

[LV] Complete missing parts of the VPlan cost model

Open
#198,721 4 comments 0 reactions 0 assignees View on GitHub
vectorizers
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

There are still some computeCosts functions in VPlan that haven't yet been filled in. I think that's because there were concerns about creating an accurate vplan cost model in case it triggered the (now deleted) legacy/vplan cost model assert. However, now the assert has gone we can start implementing these properly.

Contributor guide

Open the contributing guide

Research direction

Locate the unimplemented computeCosts functions in VPlan and read the surrounding VPlan cost-model code first. Review the context of the deleted legacy/vplan cost model assert; done means the remaining functions implement the VPlan cost model accurately without relying on that assert.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.