[PATCH] D28307: Add Instruction number to LSR cost model (PR23384) part 1 of 3

[Evgeny Stupachenko via Phabricator via llvm-commits]

evstupac added a comment.

In https://reviews.llvm.org/D28307#657004, @hfinkel wrote:

> LGTM. I assume that you're going to send a message to llvm-dev asking people to test this?


Thanks!
Yes. The change is very sensitive. For x86 results are good for benchmarks set I have, but there could be corner cases that I've missed.
Ultimate goal is to make LSR cost target depended (there will be a separate patch for this) and move InsnsCost to 1st priority for x86.


Repository:
  rL LLVM

https://reviews.llvm.org/D28307