On Tue, 29 Sep 2026 06:37:31 GMT, Quan Anh Mai <[email protected]> wrote:

>> Jatin Bhateja has refreshed the contents of this pull request, and previous 
>> commits have been removed. The incremental views will show differences 
>> compared to the previous content of the PR. The pull request contains two 
>> new commits since the last revision:
>> 
>>  - Rename vector_shift_gfni to vector_byte_shift_gfni
>>  - 8393068: Optimize uniform vector byte shift operations using x86 GFNI 
>> instruction
>
> src/hotspot/cpu/x86/c2_MacroAssembler_x86.cpp line 1372:
> 
>> 1370: }
>> 1371: 
>> 1372: void C2_MacroAssembler::vshiftb_gfni(int opcode, XMMRegister dst, 
>> XMMRegister src, XMMRegister shift,
> 
> Please add mathematical proof to this implementation.

Hi @merykitty , I have now added a comment with required details.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33113#discussion_r4141369441

Reply via email to