RFR: 8293290: RISC-V: Explicitly pass a third temp register to MacroAssembler::store_heap_oop

Aleksey Shipilev shade at openjdk.org
Fri Sep 2 12:50:41 UTC 2022


On Fri, 2 Sep 2022 03:34:49 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Currently G1 (and Shenandoah) implicitly uses x13 in oop_store_at on riscv.
> 
> This out of the blue register fixed for x86 in [JDK-8283186](https://bugs.openjdk.org/browse/JDK-8283186).
> This would be fixed in the same way on riscv by passing the temporary register explicitly so it is part of the GC API.
> 
> Testing: Passed Tier1 test on linux-riscv64 SiFive Unmatched board.

Looks fine.

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

Marked as reviewed by shade (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10137


More information about the shenandoah-dev mailing list