Integrated: 8308997: RISC-V: Sign extend when comparing 32-bit value with zero instead of testing the sign bit

[Dingli Zhang]

On Mon, 29 May 2023 03:44:28 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:

> RISC-V branch and jump instructions use 64-bit registers instead of 32-bit 
> versions in RV64I. We use test_bit to test if bit 31 is zero to determine if a
> 32-bit register is less than zero currently.
> 
> However, there will be two instructions when we use test_bit to test bit
> 31/63 without Zbs extension[1], we can just use sign_extend instead, which
> only use one instruction.
> 
> At the same time, we also change some 32-bit symbolic extension operations to
> sign_extend for better readability.
> 
> [1] https://github.com/openjdk/jdk/blob/e21f865d84c7c861843ff568019e1ad11d280a50/src/hotspot/cpu/riscv/macroAssembler_riscv.cpp#L4596-L4603
> 
> ## Testing:
> Unmatched:
> - [x] Tier1 tests (release)
> - [x] Tier2 tests (release)
> - [x] Tier3 tests (release)

This pull request has now been integrated.

Changeset: 119994f3
Author:    Dingli Zhang <dzhang at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/119994f3cedab26caa7244e49b58ab6b0b942d91
Stats:     60 lines in 10 files changed: 0 ins; 16 del; 44 mod

8308997: RISC-V: Sign extend when comparing 32-bit value with zero instead of testing the sign bit

Co-authored-by: zifeihan <caogui at iscas.ac.cn>
Reviewed-by: fjiang, fyang

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

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