Hi! My earlier attempts to replace bogus inline asm with something that reg-stack.c would be happy about failed, so this is just a quick hack to help the error-recovery on the pr84828.C testcase with -m32.
I bet bunch of other spots will need any_malformed_asm tweaks, just don't have testcases for those. Bootstrapped/regtested on x86_64-linux and i686-linux, fixes: -FAIL: g++.dg/ext/pr84828.C -std=gnu++11 (internal compiler error) -FAIL: g++.dg/ext/pr84828.C -std=gnu++11 (test for excess errors) -FAIL: g++.dg/ext/pr84828.C -std=gnu++14 (internal compiler error) -FAIL: g++.dg/ext/pr84828.C -std=gnu++14 (test for excess errors) -FAIL: g++.dg/ext/pr84828.C -std=gnu++98 (internal compiler error) -FAIL: g++.dg/ext/pr84828.C -std=gnu++98 (test for excess errors) on i686-linux, ok for trunk? 2018-04-23 Jakub Jelinek <ja...@redhat.com> PR target/84828 * reg-stack.c (move_for_stack_reg): Don't fail assertion about dead destination if any_malformed_asm. --- gcc/reg-stack.c.jj 2018-03-16 13:42:37.118852524 +0100 +++ gcc/reg-stack.c 2018-04-23 17:16:41.085198214 +0200 @@ -1105,13 +1105,16 @@ move_for_stack_reg (rtx_insn *insn, stac } /* The destination ought to be dead. */ - gcc_assert (get_hard_regnum (regstack, dest) < FIRST_STACK_REG); + if (get_hard_regnum (regstack, dest) >= FIRST_STACK_REG) + gcc_assert (any_malformed_asm); + else + { + replace_reg (psrc, get_hard_regnum (regstack, src)); - replace_reg (psrc, get_hard_regnum (regstack, src)); - - regstack->reg[++regstack->top] = REGNO (dest); - SET_HARD_REG_BIT (regstack->reg_set, REGNO (dest)); - replace_reg (pdest, FIRST_STACK_REG); + regstack->reg[++regstack->top] = REGNO (dest); + SET_HARD_REG_BIT (regstack->reg_set, REGNO (dest)); + replace_reg (pdest, FIRST_STACK_REG); + } } else if (STACK_REG_P (src)) { Jakub