From 91f66e78cc141da77ff9e0e3c8519e1af3f26c07 Mon Sep 17 00:00:00 2001 From: Segher Boessenkool Date: Fri, 24 Feb 2017 23:42:49 +0100 Subject: [PATCH] sh: Fixes for RTL checking * config/sh/sh.md (tstsi_t): If operands[0] is a SUBREG instead of a REG, look at the REG it is a SUBREG of. (splitter for cmpeqsi_t): Ditto. From-SVN: r245727 --- gcc/ChangeLog | 6 ++++++ gcc/config/sh/sh.md | 12 ++++++++++-- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 35b1609da1f..cf24e79f263 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-02-24 Segher Boessenkool + + * config/sh/sh.md (tstsi_t): If operands[0] is a SUBREG instead of + a REG, look at the REG it is a SUBREG of. + (splitter for cmpeqsi_t): Ditto. + 2017-02-24 Segher Boessenkool * config/pa/pa.c (pa_combine_instructions): Do not share RTL. Make diff --git a/gcc/config/sh/sh.md b/gcc/config/sh/sh.md index 2645fca4738..e19e977087d 100644 --- a/gcc/config/sh/sh.md +++ b/gcc/config/sh/sh.md @@ -561,8 +561,12 @@ gcc_assert (CONST_INT_P (operands[1])); HOST_WIDE_INT op1val = INTVAL (operands[1]); + rtx reg = operands[0]; + if (SUBREG_P (reg)) + reg = SUBREG_REG (reg); + gcc_assert (REG_P (reg)); bool op0_dead_after_this = - sh_reg_dead_or_unused_after_insn (curr_insn, REGNO (operands[0])); + sh_reg_dead_or_unused_after_insn (curr_insn, REGNO (reg)); if (optimize) { @@ -834,7 +838,11 @@ /* If the tested reg is not dead after this insn, it's probably used by something else after the comparison. It's probably better to leave it as it is. */ - if (find_regno_note (curr_insn, REG_DEAD, REGNO (operands[0])) == NULL_RTX) + rtx reg = operands[0]; + if (SUBREG_P (reg)) + reg = SUBREG_REG (reg); + gcc_assert (REG_P (reg)); + if (find_regno_note (curr_insn, REG_DEAD, REGNO (reg)) != NULL_RTX) FAIL; /* FIXME: Maybe also search the predecessor basic blocks to catch -- 2.30.2