RFR: 8282664: Unroll by hand StringUTF16 and StringLatin1 polynomial hash loops [v13]
Claes Redestad
redestad at openjdk.org
Tue Dec 20 20:21:57 UTC 2022
On Fri, 16 Dec 2022 23:00:53 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:
>> Claes Redestad has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Missing & 0xff in StringLatin1::hashCode
>
> src/hotspot/cpu/x86/stubRoutines_x86.cpp line 230:
>
>> 228: #endif // _LP64
>> 229:
>> 230: jint StubRoutines::x86::_arrays_hashcode_powers_of_31[] =
>
> This should be declared only for LP64.
Hmm, I guess same goes for all the new `arrays_hashcode` methods in `c2_MacroAssembler_x86`, since we only wire this up properly on 64-bit. I'll make a pass to put `_LP64` guards around all new methods.
-------------
PR: https://git.openjdk.org/jdk/pull/10847
More information about the hotspot-dev
mailing list