This is an archive of the discontinued LLVM Phabricator instance.

[VPlan] Allow planning with different cost models.
Needs ReviewPublic

Authored by fhahn on Jan 26 2023, 2:12 PM.

Details

Summary

This patch extends LVP to support planning with multiple cost-models.

This may help to simplify D142015.

Diff Detail

Event Timeline

fhahn created this revision.Jan 26 2023, 2:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2023, 2:12 PM
fhahn requested review of this revision.Jan 26 2023, 2:12 PM
fhahn updated this revision to Diff 492583.Jan 26 2023, 3:07 PM

Improve plan selection.

fhahn updated this revision to Diff 497069.Feb 13 2023, 12:13 PM

Rebase on top of D143938

arcbbb added a subscriber: arcbbb.Feb 16 2023, 7:23 AM
fhahn updated this revision to Diff 510365.Apr 2 2023, 1:46 PM

Rebased and fixed almost all test failures.

What's left is removing the remaining places where the cost-model is still used after VPlan construction.

fhahn retitled this revision from [VPlan] Allow planning with different cost models (WIP). to [VPlan] Allow planning with different cost models..
fhahn added reviewers: Ayal, gilr, dmgreen, rengolin.

This should also be in good shape now to take an initial look. There are still 2 TODOs left to resolve, but all tests expect one should pass now.

Matt added a subscriber: Matt.Apr 5 2023, 12:54 PM
fhahn updated this revision to Diff 512239.Apr 10 2023, 1:30 PM

Rebase on top of current main.

fhahn updated this revision to Diff 514015.Apr 16 2023, 8:03 AM

Rebase on top of current main.

fhahn updated this revision to Diff 516307.Apr 24 2023, 1:40 AM

rebase and ping