[PATCH] D89322: [LV] Initial VPlan cost modelling

Tue Nov 3 13:47:19 PST 2020

fhahn added a comment.

Thank you very much for putting up the patch to get things started!

I think it would be good to make sure the first steps with the cost model won't make it harder to separate the 'decision' and the 'assign cost' steps in the current cost model, in particular moving decisions out of the legacy cost model.

I tried to get started with this by moving the initial VPlan generation earlier (D90711 <https://reviews.llvm.org/D90711>) and then applying/updating this patch on top of that. I still like to experiment a bit, but I think this patch is a good first step, that would allow us to make progress in 2 directions:

1. move decisions out of the cost model
2. add VPlan transformations that require some costing.

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

https://reviews.llvm.org/D89322