RFR: 8305008: RISC-V: Factor out immediate checking functions from assembler_riscv.inline.hpp [v4]
Dingli Zhang
dzhang at openjdk.org
Thu Mar 30 08:08:11 UTC 2023
> The call-point to 'is_imm_in_range', 'is_unsigned_imm_in_range' and 'is_offset_in_range' in assembler_riscv.inline.hpp could to be replaced with shared code function 'is_simm' and 'is_uimm'.
>
> For 'is_imm_in_range', if align_bits = 0, then the call point can be changed to call 'is_simm'. If align_bits = 1, then an additional summation judgement is added to determine if it is a multiple of 2. For 'is_unsigned_imm_in_range', we can first determine if it is greater than 0, and then call 'is_uimm' further. 'is_offset_in_range' can be changed directly to call 'is_simm'. This patch also refactored for is_valid_riscv64_address and movptr.
>
> This patch removed the original functions from assembler_riscv.inline.hpp and defined some fixed-width checking functions based on the number of immediate or offset bits in the spec[1]. At the same time, we also adjusted two includes to fix the compilation error after the modification.
>
> This patch also renames the function 'operand_valid_for_add_immediate' to 'operand_valid_for_add_sub_immediate' to make it consistent with its usage.
>
> Finally, this patch modified the return value in os::non_memory_address_word to avoid an imm64 value of -1 in MacroAssembler::movptr.
>
> [1] https://github.com/riscv/riscv-isa-manual/releases/download/draft-20230131-c0b298a/riscv-spec.pdf
>
> ### Testing:
>
> On Unmatched with release build:
> - [x] Tier1 tests
> - [x] Tier2 tests
> - [x] Tier3 tests
Dingli Zhang has updated the pull request incrementally with one additional commit since the last revision:
Use is_simm12 instead
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/13202/files
- new: https://git.openjdk.org/jdk/pull/13202/files/2dfd4bc5..4470131c
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=13202&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=13202&range=02-03
Stats: 22 lines in 5 files changed: 0 ins; 7 del; 15 mod
Patch: https://git.openjdk.org/jdk/pull/13202.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/13202/head:pull/13202
PR: https://git.openjdk.org/jdk/pull/13202
More information about the hotspot-dev
mailing list