[riscv-port] RFR: 8278041: riscv: Use t2 as the dedicated machine flags register in C2 [v2]
Fei Yang
fyang at openjdk.java.net
Thu Dec 2 04:04:49 UTC 2021
On Wed, 1 Dec 2021 13:11:44 GMT, kuaiwei <duke at openjdk.java.net> wrote:
> Hi Yadong, it looks cost too much to reserve a dedicated register for control flag. I think in most cases we can use the 2 scratch registers instead. Can you show some complicated cases? We can check if they can be rewrite to avoid the complication.
Well, I think there might be some tradeoff here.
On the one hand, use the same register 't1' for two different purposes (scratch register & flags registers) at the same time looks rather error prone as mentioned in the PR description.
On the other hand, use another register 't2' as the dedicated flags register could somehow affect the performance in certain cases even though this is not reflected on the specjbb2015 numbers.
Personally, I have no obvious bias here. I would like to hear how the other Reviewers would say about this. Maybe @shipilev ?
Thanks.
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/21
More information about the riscv-port-dev
mailing list