This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
fix for gcc.c-torture/execute/990130-1
- To: egcs-patches at egcs dot cygnus dot com
- Subject: fix for gcc.c-torture/execute/990130-1
- From: Richard Henderson <rth at twiddle dot net>
- Date: Sun, 25 Apr 1999 23:08:30 -0700
Twas easier than I thought. All the uniqueness work was already
done by expand_expr.
r~
* stmt.c (expand_asm_operands): Reload in-out reg-only memory operands.
Index: stmt.c
===================================================================
RCS file: /egcs/carton/cvsfiles/egcs/gcc/stmt.c,v
retrieving revision 1.69
diff -u -p -d -r1.69 stmt.c
--- stmt.c 1999/04/04 03:06:17 1.69
+++ stmt.c 1999/04/26 06:03:02
@@ -1183,6 +1183,7 @@ expand_asm_operands (string, outputs, in
/* Vector of RTX's of evaluated output operands. */
rtx *output_rtx = (rtx *) alloca (noutputs * sizeof (rtx));
int *inout_opnum = (int *) alloca (noutputs * sizeof (int));
+ rtx *real_output_rtx = (rtx *) alloca (noutputs * sizeof (rtx));
enum machine_mode *inout_mode
= (enum machine_mode *) alloca (noutputs * sizeof (enum machine_mode));
/* The insn we have emitted. */
@@ -1360,6 +1361,7 @@ expand_asm_operands (string, outputs, in
Make the asm insn write into that, then our caller will copy it to
the real output operand. Likewise for promoted variables. */
+ real_output_rtx[i] = NULL_RTX;
if ((TREE_CODE (val) == INDIRECT_REF
&& allows_mem)
|| (TREE_CODE_CLASS (TREE_CODE (val)) == 'd'
@@ -1379,7 +1381,12 @@ expand_asm_operands (string, outputs, in
if (! allows_reg && GET_CODE (output_rtx[i]) != MEM)
error ("output number %d not directly addressable", i);
if (! allows_mem && GET_CODE (output_rtx[i]) == MEM)
- error ("output number %d not restored to memory", i);
+ {
+ real_output_rtx[i] = protect_from_queue (output_rtx[i], 1);
+ output_rtx[i] = gen_reg_rtx (GET_MODE (output_rtx[i]));
+ if (is_inout)
+ emit_move_insn (output_rtx[i], real_output_rtx[i]);
+ }
}
else
{
@@ -1661,6 +1668,12 @@ expand_asm_operands (string, outputs, in
insn = emit_insn (body);
}
+
+ /* For any outputs that needed reloading into registers, spill them
+ back to where they belong. */
+ for (i = 0; i < noutputs; ++i)
+ if (real_output_rtx[i])
+ emit_move_insn (real_output_rtx[i], output_rtx[i]);
free_temp_slots ();
}