RFR: 8302850: Implement C1 clone intrinsic that reuses arraycopy code for primitive arrays [v13]

Galder Zamarreño galder at openjdk.org
Tue Apr 30 08:43:10 UTC 2024


On Mon, 29 Apr 2024 14:34:11 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> Galder Zamarreño has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove whitespace
>
> src/hotspot/cpu/aarch64/c1_LIRAssembler_aarch64.cpp line 2508:
> 
>> 2506:  }
>> 2507: 
>> 2508:   if (stub != nullptr) {
> 
> Isn't that left over code from your previous implementation? Can `stub` be null?

I think it can be null, see https://github.com/openjdk/jdk/pull/17667/files#diff-e6f3ae4492965efd0d73c3f31073ec8b77e020740b009f92312658bac1e5f978R358

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17667#discussion_r1584371713


More information about the hotspot-compiler-dev mailing list