On Sat, 2020-02-29 at 06:16 -0800, H.J. Lu wrote:
> There is no need to set mode attribute to XImode since ix86_output_ssemov
> can properly encode xmm16-xmm31 registers with and without AVX512VL.
> 
> Remove ext_sse_reg_operand since it is no longer needed.
> 
>       PR target/89229
>       * config/i386/i386.c (ix86_output_ssemov): Handle MODE_V1DF and
>       MODE_V2SF.
>       * config/i386/mmx.md (MMXMODE:*mov<mode>_internal): Call
>       ix86_output_ssemov for TYPE_SSEMOV.  Remove ext_sse_reg_operand
>       check.
>       * config/i386/predicates.md (ext_sse_reg_operand): Removed.
This is OK.   I think once this is in, patch #2 becomes OK because this patch
adds V2SF handling in ix86_output_ssemov.

Similarly I think patch #4 is OK once this one goes in since it adds V1DF as
well.

So perhaps an integration plan would be to immediately install #6, followed 
24hrs
later by patch #4, then 24hrs after patch #2.

Then we can work on patch #5 and patch #3 where I think we go with patch #5 plus
the MODE_SI hunk from patch #3.  THen 24hrs after that the remaining bits of
patch #3.

I think that covers the whole series.

jeff

Reply via email to