[riscv-port-jdk11u:riscv-port] RFR: 8293524: RISC-V: Use macro-assembler functions as appropriate

Gui Cao gcao at openjdk.org
Tue Oct 8 06:35:23 UTC 2024


On Fri, 27 Sep 2024 03:03:41 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:

> Please review this backport to riscv-port-jdk11u.
> Backport of [JDK-8293524](https://bugs.openjdk.org/browse/JDK-8293524).
> The original patch cannot be directly applied because jdk11u has no [JDK-8241436 ](https://bugs.openjdk.org/browse/JDK-8241436 ) and [JDK-8229258](https://bugs.openjdk.org/browse/JDK-8229258) . Also jdk11u does not have generate_bigIntegerLeftShift and generate_bigIntegerRightShift functions.
> 
> Testing:
> - Tier1 passed without new failure on lp4a (release).

Why is li(result, 1); converted to mv(result, -1);?

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

> 2256: 
> 2257:   // Success
> 2258:   mv(result, -1);

why is -1?

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

> 2282:   // equal, succeed
> 2283:   if (result_as_bool) {
> 2284:     mv(result, -1);

and here.

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

> 2311: 
> 2312:   // Success
> 2313:   mv(result, -1);

and here.

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

PR Review: https://git.openjdk.org/riscv-port-jdk11u/pull/34#pullrequestreview-2353513033
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/34#discussion_r1791271129
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/34#discussion_r1791272039
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/34#discussion_r1791272298


More information about the riscv-port-dev mailing list