RFR: 8322770: Implement C2 VectorizedHashCode on AArch64 [v14]

Mikhail Ablakatov duke at openjdk.org
Wed Sep 25 16:11:20 UTC 2024


On Tue, 24 Sep 2024 15:00:58 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Mikhail Ablakatov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   cleanup: fix a comment typo
>>   
>>   Co-authored-by: Andrew Haley <aph-open at littlepinkcloud.com>
>
> src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.hpp line 39:
> 
>> 37:   // Helper functions for arrays_hashcode.
>> 38:   void arrays_hashcode_elload(Register dst, Address src, BasicType eltype);
>> 39:   int arrays_hashcode_elsize(BasicType eltype);
> 
> The above two methods don't seem to exist any more?

Fixed by https://github.com/openjdk/jdk/pull/18487/commits/0a0ab92a1e6f722bc0feaaa6787d15d734fed392.

> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 5477:
> 
>> 5475: 
>> 5476:     assert(is_power_of_2(vf), "can't use this value to calculate the jump target PC");
>> 5477:     __ andr(rscratch2, cnt, vf - 1);
> 
> It would probably be helpful to include here a repeat of the comment you added to the macroassembler method explaining how this deals with the correct number of leftover elements modulo `vf`

Fixed by https://github.com/openjdk/jdk/pull/18487/commits/142fa5d0822910ca979de60aef62ba390365afc3.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18487#discussion_r1775514892
PR Review Comment: https://git.openjdk.org/jdk/pull/18487#discussion_r1775515932


More information about the hotspot-dev mailing list