diff options
author | Bernd Schmidt <bernds@redhat.com> | 2001-05-17 17:02:35 +0000 |
---|---|---|
committer | Bernd Schmidt <bernds@gcc.gnu.org> | 2001-05-17 17:02:35 +0000 |
commit | 73b7f58c902bd97688ef3d1391039669a67ab37f (patch) | |
tree | 2b57672c9cea6ba95fc5715c3da4aa5e6c5f0e5b /gcc/expr.c | |
parent | 7332066bda945c150f7f07b6800673c68114afaa (diff) | |
download | gcc-73b7f58c902bd97688ef3d1391039669a67ab37f.zip gcc-73b7f58c902bd97688ef3d1391039669a67ab37f.tar.gz gcc-73b7f58c902bd97688ef3d1391039669a67ab37f.tar.bz2 |
Fix queueing related bugs
From-SVN: r42206
Diffstat (limited to 'gcc/expr.c')
-rw-r--r-- | gcc/expr.c | 13 |
1 files changed, 10 insertions, 3 deletions
@@ -396,6 +396,9 @@ protect_from_queue (x, modify) QUEUED_INSN (y)); return temp; } + /* Copy the address into a pseudo, so that the returned value + remains correct across calls to emit_queue. */ + XEXP (new, 0) = copy_to_reg (XEXP (new, 0)); return new; } /* Otherwise, recursively protect the subexpressions of all @@ -422,9 +425,11 @@ protect_from_queue (x, modify) } return x; } - /* If the increment has not happened, use the variable itself. */ + /* If the increment has not happened, use the variable itself. Copy it + into a new pseudo so that the value remains correct across calls to + emit_queue. */ if (QUEUED_INSN (x) == 0) - return QUEUED_VAR (x); + return copy_to_reg (QUEUED_VAR (x)); /* If the increment has happened and a pre-increment copy exists, use that copy. */ if (QUEUED_COPY (x) != 0) @@ -8588,7 +8593,9 @@ expand_expr (exp, target, tmode, modifier) if (ignore) return op0; - op0 = protect_from_queue (op0, 0); + /* Pass 1 for MODIFY, so that protect_from_queue doesn't get + clever and returns a REG when given a MEM. */ + op0 = protect_from_queue (op0, 1); /* We would like the object in memory. If it is a constant, we can have it be statically allocated into memory. For a non-constant, |