[PATCH] D118020: [RISCV] Set CostPerUse for floating point registers

Mon Mar 21 23:01:29 PDT 2022

pcwang-thead added a comment.
Herald added a subscriber: StephenFan.

In D118020#3389550 <https://reviews.llvm.org/D118020#3389550>, @asb wrote:

> @pcwang-thread: could you please post the version that just changes the preferred allocation order for review? We had a brief discussion in the RISC-V LLVM call and think that if that change is a positive improvement for you it may be simpler to just land that, initially at least. Did you have code size measurements for that too?

Thanks! I have uploaded the patch (sees D122209 <https://reviews.llvm.org/D122209>).

Code size has mild improvements under `O2` and `Oz` :
`O2`:

| Benchmark  | Code size |
| 450.soplex | -0.064%   |
| 444.namd   | -0.093%   |
| 447.dealII | -0.159%   |
| 453.povray | -0.206%   |
|

`Oz`:

| Benchmark     | Code size |
| 483.xalancbmk | -0.033%   |
| 464.h264ref   | -0.085%   |
| 447.dealII    | -0.119%   |
| 453.povray    | -0.14%    |

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118020