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

Mikhail Ablakatov duke at openjdk.org
Wed Sep 25 10:33:48 UTC 2024


On Tue, 24 Sep 2024 15:28:25 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/stubGenerator_aarch64.cpp line 5439:
> 
>> 5437:     case Assembler::T8B:
>> 5438:     case Assembler::T4H:
>> 5439:     case Assembler::T8H:
> 
> With the current code we should never see T4H as a value for load_arrangement. Is there a reason why you are folding it into the same case handling block as T8B and T8H rather than into the default block? If not then best to remove that case here and below.

The reason was to make the implementation handle all possible values of `load_arrangement` should it change. And it did while I was tuning the performance of the algorithm. The code is valid and I'd argue there's no mistake if we leave this line here. But I'm comfortable with removing it in order to make the current implementation less error-prone.

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

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


More information about the hotspot-dev mailing list