https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104502

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-9 branch has been updated by Jakub Jelinek
<ja...@gcc.gnu.org>:

https://gcc.gnu.org/g:cb412e0e881adcc5440ed37a8a415a77fe3df980

commit r9-10126-gcb412e0e881adcc5440ed37a8a415a77fe3df980
Author: Jakub Jelinek <ja...@redhat.com>
Date:   Sat Feb 12 11:17:41 2022 +0100

    i386: Fix up cvtsd2ss splitter [PR104502]

    The following testcase ICEs, because AVX512F is enabled, AVX512VL is not,
    and the cvtsd2ss insn has %xmm0-15 as output operand and %xmm16-31 as
    input operand.  For output operand %xmm16+ the splitter just gives up
    in such case, but for such input it just emits vmovddup which requires
    AVX512VL if either operand is EXT_REX_SSE_REG_P (when it is 128-bit).

    The following patch fixes it by treating that case like the pre-SSE3
    output != input case - move the input to output and do everything on
    the output reg which is known to be < %xmm16.

    2022-02-12  Jakub Jelinek  <ja...@redhat.com>

            PR target/104502
            * config/i386/i386.md (cvtsd2ss splitter): If operands[1] is xmm16+
            and AVX512VL isn't available, move operands[1] to operands[0]
first.

            * gcc.target/i386/pr104502.c: New test.

    (cherry picked from commit 0538d42cdd68f6b65d72ed7768f1d00ba44f8631)

Reply via email to