On Fri, 23 Oct 2026 22:42:13 GMT, Vladimir Kozlov <[email protected]> wrote:
>> Larry has updated the pull request incrementally with one additional commit
>> since the last revision:
>>
>> 8394259: Refine comments
>
> src/hotspot/cpu/x86/macroAssembler_x86.cpp line 8519:
>
>> 8517: vpxor(xmm1, xmm1, xmm3, Assembler::AVX_128bit);
>> 8518: vpxor(xmm1, xmm1, xmm0, Assembler::AVX_128bit);
>> 8519: if (VM_Version::supports_sse4_1()) {
>
> This is AVX code. SSE4 should be supported. I don't think we need to check
> for it. See vm_version_x86.cpp file: UseAVx will be set to 0 if UseSSE < 4.
Yes, you're right.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33342#discussion_r4377489017
