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

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

https://gcc.gnu.org/g:16dae48e9cd0421106517fc657c8743a14468945

commit r11-6562-g16dae48e9cd0421106517fc657c8743a14468945
Author: Jakub Jelinek <ja...@redhat.com>
Date:   Sat Jan 9 10:48:20 2021 +0100

    vregs: Fix up instantiate_virtual_regs_in_insn for asm goto with outputs
[PR98603]

    If an asm insn fails constraint checking during vregs, it is just deleted.
    We don't delete asm goto though because of the edges to the labels, so
    instantiate_virtual_regs_in_insn would just remove the inputs and their
    constraints, the pattern etc.
    This worked fine when asm goto couldn't have output operands, but causes
    ICEs later on when it has more than one output (and furthermore doesn't
    really remove the problematic outputs).  The problem is that
    for multiple outputs we have a PARALLEL with multiple ASM_OPERANDS, but
    those must use the same ASM_OPERANDS_INPUT_VEC etc., but the code was
    adjusting just one.

    The following patch turns invalid asm goto into a bare
    asm goto ("" : : : : lab, lab2, lab3);
    i.e. no inputs/outputs/clobbers, just the labels.

    2021-01-09  Jakub Jelinek  <ja...@redhat.com>

            PR rtl-optimization/98603
            * function.c (instantiate_virtual_regs_in_insn): For asm goto
            with impossible constraints, drop all SETs, CLOBBERs, drop PARALLEL
            if any, set ASM_OPERANDS mode to VOIDmode and change
            ASM_OPERANDS_OUTPUT_CONSTRAINT and ASM_OPERANDS_OUTPUT_IDX.

            * gcc.target/i386/pr98603.c: New test.
            * gcc.target/aarch64/pr98603.c: New test.

Reply via email to