RFR: 8295016: Make the arraycopy_epilogue signature consistent with its usage

Aleksey Shipilev shade at openjdk.org
Mon Oct 17 22:45:02 UTC 2022


On Sat, 8 Oct 2022 06:06:25 GMT, Zixian Cai <duke at openjdk.org> wrote:

> The second register argument should be named `count`. Please see the following usage.
> 
> https://github.com/openjdk/jdk/blob/495c043533d68106e07721b2e971006e9eba97e3/src/hotspot/cpu/riscv/stubGenerator_riscv.cpp#L1138
> 
> https://github.com/openjdk/jdk/blob/495c043533d68106e07721b2e971006e9eba97e3/src/hotspot/cpu/riscv/stubGenerator_riscv.cpp#L1210
> 
> https://github.com/openjdk/jdk/blob/495c043533d68106e07721b2e971006e9eba97e3/src/hotspot/cpu/riscv/stubGenerator_riscv.cpp#L1584
> 
> Update: also fix for AArch64.

This looks good to me. Bots would allow you to integrate once OCA clears.

Same problem exists in aarch64, please fix it there too?

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

Marked as reviewed by shade (Reviewer).

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


More information about the hotspot-dev mailing list