[PATCH] D9429: Improve the cost evaluation of LSR

[Quentin Colombet]

qcolombet added a comment.

Hi Wei,

Thanks for the update.

> There is no much difference between aggr-unlimited, aggr-few and aggr-estimate.


This seems to concur my feeling that the register pressure estimation of the proposed model is off.
Given that evidence, I would drop aggr-estimate and aggr-few and focus aggr-unlimited. Indeed, this is the only configuration that this new models represent correctly AFAICT.

Thanks,
-Quentin


Repository:
  rL LLVM

http://reviews.llvm.org/D9429