This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] Remove write-only assignment in rs6000-tdep.c
- From: Simon Marchi <simark at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 21 Dec 2017 01:43:19 -0000
- Subject: [binutils-gdb] Remove write-only assignment in rs6000-tdep.c
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=27e9ff87a627614fda7ddd89ddef745de5e988bb
commit 27e9ff87a627614fda7ddd89ddef745de5e988bb
Author: Uros Bizjak <ubizjak@gmail.com>
Date: Wed Dec 20 20:42:05 2017 -0500
Remove write-only assignment in rs6000-tdep.c
The result of this memory read is never used, so it can be removed.
gdb/ChangeLog:
* rs6000-tdep.c (ppc_deal_with_atomic_sequence): Remove
write-only assignment to "insn" variable.
Diff:
---
gdb/ChangeLog | 5 +++++
gdb/rs6000-tdep.c | 1 -
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index dc6406a..4f2c5c3 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2017-12-20 Uros Bizjak <ubizjak@gmail.com>
+
+ * rs6000-tdep.c (ppc_deal_with_atomic_sequence): Remove
+ write-only assignment to "insn" variable.
+
2017-12-20 Ulrich Weigand <uweigand@de.ibm.com>
* spu-tdep.c (spu_gdbarch_init): Set set_gdbarch_significant_addr_bit
diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index 456dbcc..5d0ff1d 100644
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -1226,7 +1226,6 @@ ppc_deal_with_atomic_sequence (struct regcache *regcache)
closing_insn = loc;
loc += PPC_INSN_SIZE;
- insn = read_memory_integer (loc, PPC_INSN_SIZE, byte_order);
/* Insert a breakpoint right after the end of the atomic sequence. */
breaks[0] = loc;