On Sat, 2020-02-29 at 06:16 -0800, H.J. Lu wrote:
> There is no need to set mode attribute to V16SFmode since ix86_output_ssemov
> can properly encode xmm16-xmm31 registers with and without AVX512VL.
> 
> gcc/
> 
>       PR target/89229
>       * config/i386/i386.c (ix86_output_ssemov): Handle MODE_SF.
>       * config/i386/i386.md (*movdf_internal): Call ix86_output_ssemov
>       for TYPE_SSEMOV.  Remove TARGET_PREFER_AVX256, TARGET_AVX512VL
>       and ext_sse_reg_operand check.
> 
> gcc/testsuite/
> 
>       PR target/89229
>       * gcc.target/i386/pr89229-7a.c: New test.
>       * gcc.target/i386/pr89229-7b.c: Likewise.
>       * gcc.target/i386/pr89229-7c.c: Likewise.
I believe this as a dependency on patch #3.  It's OK once patch #3 is approved. 
Alternately, you could break out the MODE_SI hunk in ix86_output_ssemov from
patch #3, add it to this patch and that would be approved for immediate
integration.

Jeff

Reply via email to