On Thu, Aug 3, 2023 at 4:16 PM Jan Beulich via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> gcc/
>
> * config/i386/sse.md
> (<avx512>_<complexopname>_<mode><maskc_name><round_name>): Add
> "prefix" attribute.
>
> (avx512fp16_<complexopname>sh_v8hf<mask_scalarc_name><round_scalarcz_name>):
> Likewise.
Ok.
> ---
> Talking of "prefix": Shouldn't at least V32HF and V32BF have it also
> default to "evex"? (It won't matter right here, but it may matter
> elsewhere.)
>
> --- a/gcc/config/i386/sse.md
> +++ b/gcc/config/i386/sse.md
> @@ -6790,6 +6790,7 @@
> return "v<complexopname><ssemodesuffix>\t{<round_maskc_op3>%2, %1,
> %0<maskc_operand3>|%0<maskc_operand3>, %1, %2<round_maskc_op3>}";
> }
> [(set_attr "type" "ssemul")
> + (set_attr "prefix" "evex")
> (set_attr "mode" "<MODE>")])
>
> (define_expand "avx512fp16_fmaddcsh_v8hf_maskz<round_expand_name>"
> @@ -6993,6 +6994,7 @@
> return "v<complexopname>sh\t{<round_scalarc_mask_op3>%2, %1,
> %0<mask_scalarc_operand3>|%0<mask_scalarc_operand3>, %1,
> %2<round_scalarc_mask_op3>}";
> }
> [(set_attr "type" "ssemul")
> + (set_attr "prefix" "evex")
> (set_attr "mode" "V8HF")])
>
> ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
>
--
BR,
Hongtao
