From: Uros Bizjak <ubizjak@gmail.com>
Date: Thu, 21 Dec 2017 01:42:05 +0000 (-0500)
Subject: Remove write-only assignment in rs6000-tdep.c
X-Git-Url: https://repo.jachan.dev/binutils.git/commitdiff_plain/27e9ff87a627614fda7ddd89ddef745de5e988bb

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 --git a/gdb/ChangeLog b/gdb/ChangeLog
index dc6406ac0c..4f2c5c33c8 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 456dbcccb1..5d0ff1d8b1 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;