On Mon, 31 Mar 2025 11:11:54 GMT, Ferenc Rakoczi <[email protected]> wrote:
>> src/hotspot/cpu/x86/vm_version_x86.cpp line 1252:
>>
>>> 1250: // Currently we only have them for AVX512
>>> 1251: #ifdef _LP64
>>> 1252: if (supports_evex() && supports_avx512bw()) {
>>
>> supports_evex check looks redundant.
>
> These are checks for two different feature bits: CPU_AVX512F and
> CPU_AVX512BW. Are you saying that the latter implies the former in every
> implementation of the spec?
AVX512BW is built on top of AVX512F spec. In assembler and other places we only
check BW in assertions which implies EVEX.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23860#discussion_r2021381288
