From 6e23f2967868d307d367385411a5e61e6de2c02b Mon Sep 17 00:00:00 2001 From: Vladimir Makarov Date: Thu, 23 Jan 2014 20:06:28 +0000 Subject: [PATCH] re PR rtl-optimization/59915 (LRA ICE - Repeated looping over subreg reloads (gcc.c-torture/compile/simd-3.c)) 2014-01-23 Vladimir Makarov PR regression/59915 * lra-constraints.c (simplify_operand_subreg): Spill pseudo if there is a danger of looping. From-SVN: r207007 --- gcc/ChangeLog | 6 ++++++ gcc/lra-constraints.c | 17 ++++++++++++++--- 2 files changed, 20 insertions(+), 3 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2998c727844..874352c08a8 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2014-01-23 Vladimir Makarov + + PR regression/59915 + * lra-constraints.c (simplify_operand_subreg): Spill pseudo if + there is a danger of looping. + 2014-01-23 Pat Haugen * config/rs6000/rs6000.c (rs6000_option_override_internal): Don't diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c index 7454229a7d6..34159f75a10 100644 --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -1291,9 +1291,20 @@ simplify_operand_subreg (int nop, enum machine_mode reg_mode) && ! LRA_SUBREG_P (operand)) || CONSTANT_P (reg) || GET_CODE (reg) == PLUS || MEM_P (reg)) { - /* The class will be defined later in curr_insn_transform. */ - enum reg_class rclass - = (enum reg_class) targetm.preferred_reload_class (reg, ALL_REGS); + enum reg_class rclass; + + if (REG_P (reg) + && curr_insn_set != NULL_RTX + && (REG_P (SET_SRC (curr_insn_set)) + || GET_CODE (SET_SRC (curr_insn_set)) == SUBREG)) + /* There is big probability that we will get the same class + for the new pseudo and we will get the same insn which + means infinite looping. So spill the new pseudo. */ + rclass = NO_REGS; + else + /* The class will be defined later in curr_insn_transform. */ + rclass + = (enum reg_class) targetm.preferred_reload_class (reg, ALL_REGS); if (get_reload_reg (curr_static_id->operand[nop].type, reg_mode, reg, rclass, "subreg reg", &new_reg))