RFR: 8296602: RISC-V: improve performance of copy_memory stub [v2]

Fei Yang fyang at openjdk.org
Thu Nov 17 08:21:40 UTC 2022


On Thu, 17 Nov 2022 08:05:59 GMT, Vladimir Kempik <vkempik at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/stubGenerator_riscv.cpp line 1042:
>> 
>>> 1040:     __ bind(copy32);
>>> 1041:     if (is_backwards) {
>>> 1042:       __ addi(src, src, -wordSize*4);
>> 
>> Can you leave a space before and after the operator here and other places?
>
> Not sure I'm getting you right here. Please elaborate

I am suggesting this style:

__ addi(src, src, -wordSize * 4);

instead of:

__ addi(src, src, -wordSize*4);

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

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


More information about the hotspot-compiler-dev mailing list