RFR: 8374351: RISC-V: Small refactoring for crypto macro-assembler routines [v2]

Anjian Wen wenanjian at openjdk.org
Fri Dec 26 03:06:37 UTC 2025


On Fri, 26 Dec 2025 02:43:35 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Anjian Wen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Swap the order of the function name to make it clear
>
> src/hotspot/cpu/riscv/stubGenerator_riscv.cpp line 3059:
> 
>> 3057:     VectorRegister vtmp3 = v3;
>> 3058: 
>> 3059:     ghash_loop(subkeyH, state, data, blocks, vtmp1, vtmp2, vtmp3);
> 
> `state` is the first param for `generate_ghash_processBlocks`. Can we simply swap the first two params to keep that order?

sure, Thanks for the advice

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28988#discussion_r2647433037


More information about the hotspot-compiler-dev mailing list