Commit 4a5d0fb5 by Richard Stallman

(eliminate_regs_in_insn): Rerecognize if move insn becomes a non-move.

From-SVN: r2257
parent a6b65dff
...@@ -2987,9 +2987,21 @@ eliminate_regs_in_insn (insn, replace) ...@@ -2987,9 +2987,21 @@ eliminate_regs_in_insn (insn, replace)
new_body = eliminate_regs (old_body, 0, replace ? insn : NULL_RTX); new_body = eliminate_regs (old_body, 0, replace ? insn : NULL_RTX);
if (new_body != old_body) if (new_body != old_body)
{ {
if (GET_CODE (old_body) != SET || GET_CODE (SET_SRC (old_body)) != PLUS /* If we had a move insn but now we don't, rerecognize it. */
|| ! validate_change (insn, &PATTERN (insn), new_body, 0)) if (GET_CODE (old_body) == SET && GET_CODE (SET_SRC (old_body)) == REG
&& (GET_CODE (new_body) != SET
|| GET_CODE (SET_SRC (new_body)) != REG))
{
if (! validate_change (insn, &PATTERN (insn), new_body, 0))
abort ();
}
/* If this was not a move insn, rerecognize. */
else if (GET_CODE (old_body) != SET
|| GET_CODE (SET_SRC (old_body)) != PLUS
|| ! validate_change (insn, &PATTERN (insn), new_body, 0))
PATTERN (insn) = new_body; PATTERN (insn) = new_body;
/* ??? Is it really correct to store the new body anyway
if validate_change fails? Shouldn't this abort instead? */
if (replace && REG_NOTES (insn)) if (replace && REG_NOTES (insn))
REG_NOTES (insn) = eliminate_regs (REG_NOTES (insn), 0, NULL_RTX); REG_NOTES (insn) = eliminate_regs (REG_NOTES (insn), 0, NULL_RTX);
......
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