On Fri, 16 Dec 2022 23:00:53 GMT, Sandhya Viswanathan 
<sviswanat...@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

Reply via email to