On Wed, 8 Apr 2026 10:09:43 GMT, Jatin Bhateja <[email protected]> wrote:
>> src/hotspot/cpu/x86/x86.ad line 3418:
>>
>>> 3416: ((size_in_bits != 512) &&
>>> !VM_Version::supports_avx512vl()))) {
>>> 3417: return false;
>>> 3418: }
>>
>> This can be simplified to:
>>
>> if (UseAVX > 2 && !VM_Version::supports_avx512vlbw()) {
>> return false;
>> }
>>
>> As the platforms supporting bw also support vl.
>
> It was segregated because for size_in_bits set to 512 we don't need AVX512VL.
In the instruct rules below the predicate is always handling avx512vlbw
together. There is no separate consideration for avx512bw alone. e.g. the
512-bit vector width in such case goes to vector_slice_avx which would assert.
As there is no such platform that supports bw and doesn't support vl there is
no point in adding complications.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24104#discussion_r3052804443
