This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[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;


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]