Re: [PATCH] Correct cpu_flags for CPU_ANY_AVX512BW_FLAGS

[Jan Beulich <jbeulich@suse.com>]

On 23.06.2022 09:14, Cui,Lili wrote:
> This patch is to correct cpu_flags for CPU_ANY_AVX512BW_FLAGS.
> Make check-gas is ok.
> Ok for master?
> 
> Thanks,
> Lili.
> 
> Change it from "CPU_ANY_AVX512_FP16_FLAGS" to "CpuAVX512_FP16"

You don't say what's wrong with what there is right now. I for one
think what we have is correct: If one disables BW, _all_ its
dependents should be disabled as well, which transitively means all
dependents of FP16.

Jan

> opcodes/
> 
> 	* i386-gen.c (cpu_flag_init): Change cpu_flags of
> 	CPU_ANY_AVX512BW_FLAGS.
> ---
>  opcodes/i386-gen.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/opcodes/i386-gen.c b/opcodes/i386-gen.c
> index d18a7d2754..5dc224efdb 100644
> --- a/opcodes/i386-gen.c
> +++ b/opcodes/i386-gen.c
> @@ -382,7 +382,7 @@ static initializer cpu_flag_init[] =
>    { "CPU_ANY_AVX512DQ_FLAGS",
>      "CpuAVX512DQ" },
>    { "CPU_ANY_AVX512BW_FLAGS",
> -    "CpuAVX512BW|CPU_ANY_AVX512_FP16_FLAGS" },
> +    "CpuAVX512BW|CpuAVX512_FP16" },
>    { "CPU_ANY_AVX512VL_FLAGS",
>      "CpuAVX512VL" },
>    { "CPU_ANY_AVX512IFMA_FLAGS",