RFR: 8320500: [vectorapi] RISC-V: Optimize vector math operations with SLEEF [v4]

[Fei Yang]

On Tue, 24 Sep 2024 05:25:38 GMT, Robbin Ehn <rehn at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/assembler_riscv.hpp line 51:
>> 
>>> 49:     n_int_register_parameters_c   = 8,   // x10, x11, ... x17 (c_rarg0, c_rarg1, ...)
>>> 50:     n_float_register_parameters_c = 8,   // f10, f11, ... f17 (c_farg0, c_farg1, ... )
>>> 51:     n_vector_register_parameters_c = 8,  // v8, v9, ... v15
>> 
>> I know vector registers are not used for passing arguments or return values by the RISCV ABI for now. But I guess it might be better and consistent to align with the numbering of integer and floating-point argument registers (x10 - x17, f10 - f17)? That is v10 - v17.
>
> Note in the RISC-V ELF psABI there is a convetion variant for v-regs.
> If you add function attribute riscv_vector_cc it should be used for C/C++. (I never tested it)
> v0 = first vector mask argument
> v8-v23 = args/rets
> v1-v7/v24-v31 = caller saved
> 
> https://github.com/riscv-non-isa/riscv-elf-psabi-doc/blob/master/riscv-cc.adoc

Ah, I think I missed that. I was reading psABI spec 1.0 release. Thanks for this info.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21083#discussion_r1772742406