Commit 112f7223 by Ulrich Weigand Committed by Ulrich Weigand

* reload.c: Revert 2003-01-31 change.

From-SVN: r62249
parent 064994a3
2003-02-01 Ulrich Weigand <uweigand@de.ibm.com>
* reload.c: Revert 2003-01-31 change.
Sat Feb 1 14:14:40 CET 2003 Jan Hubicka <jh@suse.cz> Sat Feb 1 14:14:40 CET 2003 Jan Hubicka <jh@suse.cz>
* combine.c (combine_simplify_rtx): Use reversed_comparison_code_parts. * combine.c (combine_simplify_rtx): Use reversed_comparison_code_parts.
......
...@@ -274,7 +274,6 @@ static void find_reloads_address_part PARAMS ((rtx, rtx *, enum reg_class, ...@@ -274,7 +274,6 @@ static void find_reloads_address_part PARAMS ((rtx, rtx *, enum reg_class,
static rtx find_reloads_subreg_address PARAMS ((rtx, int, int, static rtx find_reloads_subreg_address PARAMS ((rtx, int, int,
enum reload_type, int, rtx)); enum reload_type, int, rtx));
static void copy_replacements_1 PARAMS ((rtx *, rtx *, int)); static void copy_replacements_1 PARAMS ((rtx *, rtx *, int));
static bool have_replacement_p PARAMS ((rtx *));
static int find_inc_amount PARAMS ((rtx, rtx)); static int find_inc_amount PARAMS ((rtx, rtx));
#ifdef HAVE_SECONDARY_RELOADS #ifdef HAVE_SECONDARY_RELOADS
...@@ -2690,10 +2689,6 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p) ...@@ -2690,10 +2689,6 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p)
recog_data.operand[i] = *recog_data.operand_loc[i]; recog_data.operand[i] = *recog_data.operand_loc[i];
substed_operand[i] = recog_data.operand[i]; substed_operand[i] = recog_data.operand[i];
/* Address operands are reloaded in their existing mode,
no matter what is specified in the machine description. */
operand_mode[i] = GET_MODE (recog_data.operand[i]);
} }
else if (code == MEM) else if (code == MEM)
{ {
...@@ -3307,6 +3302,10 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p) ...@@ -3307,6 +3302,10 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p)
the address into a base register. */ the address into a base register. */
this_alternative[i] = (int) MODE_BASE_REG_CLASS (VOIDmode); this_alternative[i] = (int) MODE_BASE_REG_CLASS (VOIDmode);
badop = 0; badop = 0;
/* Address constraints are reloaded in Pmode, no matter
what mode is given in the machine description. */
operand_mode[i] = Pmode;
break; break;
} }
...@@ -3877,20 +3876,9 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p) ...@@ -3877,20 +3876,9 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p)
return 0; return 0;
} }
} }
/* Generate optional reloads only when optimizing, and only
on the last pass through reload. Also, make sure we do not
make an optional reload where we already have a mandatory
one; this can happen in the case of address operands.
To check for mandatory reloads, we use have_replacement_p.
Note that this works only on the last pass through reload. */
else if (!optimize || !replace
|| have_replacement_p (recog_data.operand_loc[i]))
; /* Do nothing. */
else if (goal_alternative_matched[i] < 0 else if (goal_alternative_matched[i] < 0
&& goal_alternative_matches[i] < 0) && goal_alternative_matches[i] < 0
&& optimize)
{ {
/* For each non-matching operand that's a MEM or a pseudo-register /* For each non-matching operand that's a MEM or a pseudo-register
that didn't get a hard register, make an optional reload. that didn't get a hard register, make an optional reload.
...@@ -3940,10 +3928,11 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p) ...@@ -3940,10 +3928,11 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p)
reload, check if this is actually a pseudo register reference; reload, check if this is actually a pseudo register reference;
we then need to emit a USE and/or a CLOBBER so that reload we then need to emit a USE and/or a CLOBBER so that reload
inheritance will do the right thing. */ inheritance will do the right thing. */
else if (GET_CODE (operand) == MEM else if (replace
|| (GET_CODE (operand) == REG && (GET_CODE (operand) == MEM
&& REGNO (operand) >= FIRST_PSEUDO_REGISTER || (GET_CODE (operand) == REG
&& reg_renumber [REGNO (operand)] < 0)) && REGNO (operand) >= FIRST_PSEUDO_REGISTER
&& reg_renumber [REGNO (operand)] < 0)))
{ {
operand = *recog_data.operand_loc[i]; operand = *recog_data.operand_loc[i];
...@@ -3966,7 +3955,8 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p) ...@@ -3966,7 +3955,8 @@ find_reloads (insn, replace, ind_levels, live_known, reload_reg_p)
&& goal_alternative_win[goal_alternative_matches[i]] && goal_alternative_win[goal_alternative_matches[i]]
&& modified[i] == RELOAD_READ && modified[i] == RELOAD_READ
&& modified[goal_alternative_matches[i]] == RELOAD_WRITE && modified[goal_alternative_matches[i]] == RELOAD_WRITE
&& ! no_input_reloads && ! no_output_reloads) && ! no_input_reloads && ! no_output_reloads
&& optimize)
{ {
/* Similarly, make an optional reload for a pair of matching /* Similarly, make an optional reload for a pair of matching
objects that are in MEM or a pseudo that didn't get a hard reg. */ objects that are in MEM or a pseudo that didn't get a hard reg. */
...@@ -6140,21 +6130,6 @@ find_replacement (loc) ...@@ -6140,21 +6130,6 @@ find_replacement (loc)
return *loc; return *loc;
} }
/* Return true if some replacement was scheduled at LOC. */
static bool
have_replacement_p (loc)
rtx *loc;
{
struct replacement *r;
for (r = &replacements[0]; r < &replacements[n_replacements]; r++)
if (r->where == loc)
return true;
return false;
}
/* Return nonzero if register in range [REGNO, ENDREGNO) /* Return nonzero if register in range [REGNO, ENDREGNO)
appears either explicitly or implicitly in X appears either explicitly or implicitly in X
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment