Commit 1e32ce0b by Sandra Loosemore Committed by Sandra Loosemore

c6x.c (try_rename_operands): Do not depend on gcc_assert evaluating its argument for side-effect.

2015-06-30  Sandra Loosemore <sandra@codesourcery.com>

	gcc/
	* config/c6x/c6x.c (try_rename_operands): Do not depend on
	gcc_assert evaluating its argument for side-effect.

From-SVN: r225221
parent f58d5e06
2015-06-30 Sandra Loosemore <sandra@codesourcery.com>
* config/c6x/c6x.c (try_rename_operands): Do not depend on
gcc_assert evaluating its argument for side-effect.
2015-06-30 Kaz Kojima <kkojima@gcc.gnu.org>
PR target/64833
......
......@@ -3450,6 +3450,7 @@ try_rename_operands (rtx_insn *head, rtx_insn *tail, unit_req_table reqs,
int best_reg, old_reg;
vec<du_head_p> involved_chains = vNULL;
unit_req_table new_reqs;
bool ok;
for (i = 0, tmp_mask = op_mask; tmp_mask; i++)
{
......@@ -3516,7 +3517,8 @@ try_rename_operands (rtx_insn *head, rtx_insn *tail, unit_req_table reqs,
best_reg =
find_rename_reg (this_head, super_class, &unavailable, old_reg, true);
gcc_assert (regrename_do_replace (this_head, best_reg));
ok = regrename_do_replace (this_head, best_reg);
gcc_assert (ok);
count_unit_reqs (new_reqs, head, PREV_INSN (tail));
merge_unit_reqs (new_reqs);
......@@ -3529,7 +3531,10 @@ try_rename_operands (rtx_insn *head, rtx_insn *tail, unit_req_table reqs,
unit_req_imbalance (reqs), unit_req_imbalance (new_reqs));
}
if (unit_req_imbalance (new_reqs) > unit_req_imbalance (reqs))
gcc_assert (regrename_do_replace (this_head, old_reg));
{
ok = regrename_do_replace (this_head, old_reg);
gcc_assert (ok);
}
else
memcpy (reqs, new_reqs, sizeof (unit_req_table));
......
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