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

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 14 10:59:47 PDT 2017


qcolombet requested changes to this revision.
qcolombet added a comment.
This revision now requires changes to proceed.

Hi,

I see two different things in that patch:

1. X86 cost model change
2. GenerateCrossUseICmpZero

Please make two separate patches.

For both cases, I expect a thorough benchmarking numbers for Os and https://reviews.llvm.org/owners/package/3/.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D30562





More information about the llvm-commits mailing list