summaryrefslogtreecommitdiff
path: root/gcc/reload.c
diff options
context:
space:
mode:
authorhp <hp@138bc75d-0d04-0410-961f-82ee72b054a4>2001-08-25 21:39:47 +0000
committerhp <hp@138bc75d-0d04-0410-961f-82ee72b054a4>2001-08-25 21:39:47 +0000
commitf6ada591fc93dacb66c99f70c7317f3eb984c049 (patch)
tree3f04cc547069c87b98593d538db187624b02a7b5 /gcc/reload.c
parentf4c0c1a259e6ba8b205f9599fb876d9f0780d2d8 (diff)
downloadgcc-f6ada591fc93dacb66c99f70c7317f3eb984c049.tar.gz
* reload1.c (reload): Make all entries in reg_equiv_memory_loc
unshared. * reload.c (make_memloc): Copy result if it is still reg_equiv_memory_loc[regno] on return. (subst_reloads) [ENABLE_CHECKING]: Check that none of reg_equiv_constant, reg_equiv_memory_loc, reg_equiv_address and reg_equiv_mem are modified by the substitutions. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@45177 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc/reload.c')
-rw-r--r--gcc/reload.c34
1 files changed, 33 insertions, 1 deletions
diff --git a/gcc/reload.c b/gcc/reload.c
index 73f8b2185ec..0cf5a0eebf9 100644
--- a/gcc/reload.c
+++ b/gcc/reload.c
@@ -4486,7 +4486,13 @@ make_memloc (ad, regno)
tem = copy_rtx (tem);
tem = replace_equiv_address_nv (reg_equiv_memory_loc[regno], tem);
- return adjust_address_nv (tem, GET_MODE (ad), 0);
+ tem = adjust_address_nv (tem, GET_MODE (ad), 0);
+
+ /* Copy the result if it's still the same as the equivalence, to avoid
+ modifying it when we do the substitution for the reload. */
+ if (tem == reg_equiv_memory_loc[regno])
+ tem = copy_rtx (tem);
+ return tem;
}
/* Record all reloads needed for handling memory address AD
@@ -5769,6 +5775,32 @@ subst_reloads (insn)
register rtx reloadreg = rld[r->what].reg_rtx;
if (reloadreg)
{
+#ifdef ENABLE_CHECKING
+ /* Internal consistency test. Check that we don't modify
+ anything in the equivalence arrays. Whenever something from
+ those arrays needs to be reloaded, it must be unshared before
+ being substituted into; the equivalence must not be modified.
+ Otherwise, if the equivalence is used after that, it will
+ have been modified, and the thing substituted (probably a
+ register) is likely overwritten and not a usable equivalence. */
+ int check_regno;
+
+ for (check_regno = 0; check_regno < max_regno; check_regno++)
+ {
+#define CHECK_MODF(ARRAY) \
+ if (ARRAY[check_regno] \
+ && loc_mentioned_in_p (r->where, \
+ ARRAY[check_regno])) \
+ abort ()
+
+ CHECK_MODF (reg_equiv_constant);
+ CHECK_MODF (reg_equiv_memory_loc);
+ CHECK_MODF (reg_equiv_address);
+ CHECK_MODF (reg_equiv_mem);
+#undef CHECK_MODF
+ }
+#endif /* ENABLE_CHECKING */
+
/* If we're replacing a LABEL_REF with a register, add a
REG_LABEL note to indicate to flow which label this
register refers to. */