On Wed, Nov 27, 2019 at 11:32:24AM +0800, Hongtao Liu wrote:
> hi jakub:
> VF is used for differentiating AVX512F/AVX/SSE, but there's
> condition TARGET_AVX512F in avx512f_maskcmp<mode>3, it must be a TYPO
> and should be VF_AVX512VL instead.
>
> Bootstrap and regression test on i386/x86_64 backend is ok.
> OK for trunk?
Ok, thanks.
> diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md
> index 62c68053563..fb43cafaad0 100644
> --- a/gcc/config/i386/sse.md
> +++ b/gcc/config/i386/sse.md
> @@ -3129,8 +3129,8 @@
> (define_insn "avx512f_maskcmp<mode>3"
> [(set (match_operand:<avx512fmaskmode> 0 "register_operand" "=k")
> (match_operator:<avx512fmaskmode> 3 "sse_comparison_operator"
> - [(match_operand:VF 1 "register_operand" "v")
> - (match_operand:VF 2 "nonimmediate_operand" "vm")]))]
> + [(match_operand:VF_AVX512VL 1 "register_operand" "v")
> + (match_operand:VF_AVX512VL 2 "nonimmediate_operand" "vm")]))]
> "TARGET_AVX512F"
> "vcmp%D3<ssemodesuffix>\t{%2, %1, %0|%0, %1, %2}"
> [(set_attr "type" "ssecmp")
Jakub
