From 7877f4e000e999a902cf2898a535ae8e8ad45a3f Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Tue, 13 Nov 2018 14:25:30 +1030 Subject: [PATCH] [RS6000] secondary_reload and find_replacement This patch removes a call only necessary when using reload. It also corrects a PRE_DEC address offset. * config/rs6000/rs6000.c (rs6000_secondary_reload_inner): Negate offset for PRE_DEC. (rs6000_secondary_reload_gpr): Don't call find_replacement. From-SVN: r266049 --- gcc/ChangeLog | 6 ++++++ gcc/config/rs6000/rs6000.c | 16 ++++------------ 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 34b0726..3f98c90 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2018-11-13 Alan Modra + + * config/rs6000/rs6000.c (rs6000_secondary_reload_inner): Negate + offset for PRE_DEC. + (rs6000_secondary_reload_gpr): Don't call find_replacement. + 2018-11-13 Sandra Loosemore PR middle-end/59634 diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index e0b1bbb..516e697 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -19701,7 +19701,10 @@ rs6000_secondary_reload_inner (rtx reg, rtx mem, rtx scratch, bool store_p) if ((addr_mask & RELOAD_REG_PRE_INCDEC) == 0) { - emit_insn (gen_add2_insn (op_reg, GEN_INT (GET_MODE_SIZE (mode)))); + int delta = GET_MODE_SIZE (mode); + if (GET_CODE (addr) == PRE_DEC) + delta = -delta; + emit_insn (gen_add2_insn (op_reg, GEN_INT (delta))); new_addr = op_reg; } break; @@ -19901,17 +19904,6 @@ rs6000_secondary_reload_gpr (rtx reg, rtx mem, rtx scratch, bool store_p) && GET_CODE (XEXP (addr, 1)) == PLUS && XEXP (XEXP (addr, 1), 0) == XEXP (addr, 0)); scratch_or_premodify = XEXP (addr, 0); - if (!HARD_REGISTER_P (scratch_or_premodify)) - /* If we have a pseudo here then reload will have arranged - to have it replaced, but only in the original insn. - Use the replacement here too. */ - scratch_or_premodify = find_replacement (&XEXP (addr, 0)); - - /* RTL emitted by rs6000_secondary_reload_gpr uses RTL - expressions from the original insn, without unsharing them. - Any RTL that points into the original insn will of course - have register replacements applied. That is why we don't - need to look for replacements under the PLUS. */ addr = XEXP (addr, 1); } gcc_assert (GET_CODE (addr) == PLUS || GET_CODE (addr) == LO_SUM); -- 2.7.4