[riscv-port] RFR: 8278192: riscv: remove unnecessary instruct of DecodeNKlass in C2

Fei Yang fyang at openjdk.java.net
Fri Dec 3 10:48:42 UTC 2021


On Fri, 3 Dec 2021 08:20:17 GMT, Yadong Wang <yadongwang at openjdk.org> wrote:

> There are two instructs for DecodeNKlass in C2, and the difference is only that which temporary register operand is used by them, xheapbase, t0 or t1. It's too complicated and the effect is almost invisible. So we just simplify the pattern by using a temporary register from the register allocation.

Changes requested by fyang (Lead).

src/hotspot/cpu/riscv/macroAssembler_riscv.cpp line 2202:

> 2200:   Label retry_load, nope;
> 2201:   bind(retry_load);
> 2202:   // flush and load exclusive from the memory location

Could you please also fix the comments in MacroAssembler::atomic_incw? Thanks

-------------

PR: https://git.openjdk.java.net/riscv-port/pull/22


More information about the riscv-port-dev mailing list