RFR: 8349452: Fix performance regression for Arrays.fill() with AVX512 [v2]

Srinivas Vamsi Parasa sparasa at openjdk.org
Sat Nov 22 00:03:49 UTC 2025


On Fri, 21 Nov 2025 22:00:12 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> Srinivas Vamsi Parasa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   undo size check for fill64_masked
>
> src/hotspot/cpu/x86/macroAssembler_x86.cpp line 9380:
> 
>> 9378:     bind(L_fill_96_bytes);
>> 9379:     cmpq(count, 96 >> shift);
>> 9380:     jcc(Assembler::greater, L_fill_128_bytes);
> 
> With the suggestion to have fill32_unmasked and fill64_unmasked one time, you may be able to retain the jccb.

The jccb to jcc was undone as needed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28442#discussion_r2551311201


More information about the hotspot-dev mailing list