This is the mail archive of the
mailing list for the GCC project.
[patch] explow.c: Remove stabilize and its subroutine.
- From: Kazu Hirata <kazu at cs dot umass dot edu>
- To: gcc-patches at gcc dot gnu dot org
- Date: Fri, 14 Jan 2005 14:58:31 -0500 (EST)
- Subject: [patch] explow.c: Remove stabilize and its subroutine.
Attached is a patch to remove stabilize and copy_all_regs as they are
AFAICT, the last use of stabilize was removed in
Even then the last use was surrounded by #if 0, so the function has
been unused for at least half a year.
Note that copy_all_regs is a subroutine of stabilize.
Bootstrapped on i686-pc-linux-gnu. OK to apply?
2005-01-14 Kazu Hirata <firstname.lastname@example.org>
* explow.c (copy_all_reg, stabilize): Remove.
* expr.h: Remove the corresponding prototypes.
RCS file: /cvs/gcc/gcc/gcc/explow.c,v
retrieving revision 1.141
diff -u -d -p -r1.141 explow.c
--- explow.c 11 Nov 2004 23:14:20 -0000 1.141
+++ explow.c 14 Jan 2005 08:46:24 -0000
@@ -390,44 +390,6 @@ convert_memory_address (enum machine_mod
#endif /* defined(POINTERS_EXTEND_UNSIGNED) */
-/* Given a memory address or facsimile X, construct a new address,
- currently equivalent, that is stable: future stores won't change it.
- X must be composed of constants, register and memory references
- combined with addition, subtraction and multiplication:
- in other words, just what you can get from expand_expr if sum_ok is 1.
- Works by making copies of all regs and memory locations used
- by X and combining them the same way X does.
- You could also stabilize the reference to this address
- by copying the address to a register with copy_to_reg;
- but then you wouldn't get indexed addressing in the reference. */
-copy_all_regs (rtx x)
- if (REG_P (x))
- if (REGNO (x) != FRAME_POINTER_REGNUM
-#if HARD_FRAME_POINTER_REGNUM != FRAME_POINTER_REGNUM
- && REGNO (x) != HARD_FRAME_POINTER_REGNUM
- x = copy_to_reg (x);
- else if (MEM_P (x))
- x = copy_to_reg (x);
- else if (GET_CODE (x) == PLUS || GET_CODE (x) == MINUS
- || GET_CODE (x) == MULT)
- rtx op0 = copy_all_regs (XEXP (x, 0));
- rtx op1 = copy_all_regs (XEXP (x, 1));
- if (op0 != XEXP (x, 0) || op1 != XEXP (x, 1))
- x = gen_rtx_fmt_ee (GET_CODE (x), Pmode, op0, op1);
- return x;
/* Return something equivalent to X but valid as a memory address
for something of mode MODE. When X is not itself valid, this
@@ -576,22 +538,6 @@ validize_mem (rtx ref)
return replace_equiv_address (ref, XEXP (ref, 0));
-/* Return a modified copy of X with its memory address copied
- into a temporary register to protect it from side effects.
- If X is not a MEM, it is returned unchanged (and not copied).
- Perhaps even if it is a MEM, if there is no need to change it. */
-stabilize (rtx x)
- if (!MEM_P (x)
- || ! rtx_unstable_p (XEXP (x, 0)))
- return x;
- replace_equiv_address (x, force_reg (Pmode, copy_all_regs (XEXP (x, 0))));
/* Copy the value or contents of X to a new temp reg and return that reg. */
RCS file: /cvs/gcc/gcc/gcc/expr.h,v
retrieving revision 1.185
diff -u -d -p -r1.185 expr.h
--- expr.h 11 Jan 2005 09:51:16 -0000 1.185
+++ expr.h 14 Jan 2005 08:46:24 -0000
@@ -642,14 +642,6 @@ extern void set_mem_attributes_minus_bit
/* Assemble the static constant template for function entry trampolines. */
extern rtx assemble_trampoline_template (void);
-/* Given rtx, return new rtx whose address won't be affected by
- any side effects. It has been copied to a new temporary reg. */
-extern rtx stabilize (rtx);
-/* Given an rtx, copy all regs it refers to into new temps
- and return a modified copy that refers to the new temps. */
-extern rtx copy_all_regs (rtx);
/* Copy given rtx to a new temp reg and return that. */
extern rtx copy_to_reg (rtx);