aboutsummaryrefslogtreecommitdiff
path: root/gcc/flow.c
diff options
context:
space:
mode:
authorRichard Henderson <rth@redhat.com>2000-11-08 16:26:34 -0800
committerRichard Henderson <rth@gcc.gnu.org>2000-11-08 16:26:34 -0800
commitc32e1e6fbc96ee261728dbbc5da6fd69619b15b8 (patch)
tree424ab1377c7f1bbb66c3532a2d7d64391c87ef02 /gcc/flow.c
parentadb84b4254d23c4ff58f53c4f48971b6cd8395d1 (diff)
downloadgcc-c32e1e6fbc96ee261728dbbc5da6fd69619b15b8.zip
gcc-c32e1e6fbc96ee261728dbbc5da6fd69619b15b8.tar.gz
gcc-c32e1e6fbc96ee261728dbbc5da6fd69619b15b8.tar.bz2
flow.c (init_propagate_block_info): Protect the rtx stored in mem_set_list from modification by find_auto_inc.
* flow.c (init_propagate_block_info): Protect the rtx stored in mem_set_list from modification by find_auto_inc. (mark_set_1): Likewise. From-SVN: r37330
Diffstat (limited to 'gcc/flow.c')
-rw-r--r--gcc/flow.c26
1 files changed, 24 insertions, 2 deletions
diff --git a/gcc/flow.c b/gcc/flow.c
index d286f91..b855114 100644
--- a/gcc/flow.c
+++ b/gcc/flow.c
@@ -3930,7 +3930,21 @@ init_propagate_block_info (bb, live, local_set, flags)
|| (GET_CODE (XEXP (mem, 0)) == PLUS
&& XEXP (XEXP (mem, 0), 0) == frame_pointer_rtx
&& GET_CODE (XEXP (XEXP (mem, 0), 1)) == CONST_INT))
- pbi->mem_set_list = alloc_EXPR_LIST (0, mem, pbi->mem_set_list);
+ {
+#ifdef AUTO_INC_DEC
+ /* Store a copy of mem, otherwise the address may be scrogged
+ by find_auto_inc. This matters because insn_dead_p uses
+ an rtx_equal_p check to determine if two addresses are
+ the same. This works before find_auto_inc, but fails
+ after find_auto_inc, causing discrepencies between the
+ set of live registers calculated during the
+ calculate_global_regs_live phase and what actually exists
+ after flow completes, leading to aborts. */
+ if (flags & PROP_AUTOINC)
+ mem = shallow_copy_rtx (mem);
+#endif
+ pbi->mem_set_list = alloc_EXPR_LIST (0, mem, pbi->mem_set_list);
+ }
}
}
@@ -4561,7 +4575,15 @@ mark_set_1 (pbi, code, reg, cond, insn, flags)
everything that invalidates it. To be safe, don't eliminate any
stores though SP; none of them should be redundant anyway. */
&& ! reg_mentioned_p (stack_pointer_rtx, reg))
- pbi->mem_set_list = alloc_EXPR_LIST (0, reg, pbi->mem_set_list);
+ {
+#ifdef AUTO_INC_DEC
+ /* Store a copy of mem, otherwise the address may be
+ scrogged by find_auto_inc. */
+ if (flags & PROP_AUTOINC)
+ reg = shallow_copy_rtx (reg);
+#endif
+ pbi->mem_set_list = alloc_EXPR_LIST (0, reg, pbi->mem_set_list);
+ }
}
if (GET_CODE (reg) == REG