This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] sparc64-tdep.c: Don't assign using memcpy return
- From: Simon Marchi <simark at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 24 Oct 2015 22:38:13 -0000
- Subject: [binutils-gdb] sparc64-tdep.c: Don't assign using memcpy return
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=cfcb22a541ab8f783bf667cf7afbbfd9cd243cce
commit cfcb22a541ab8f783bf667cf7afbbfd9cd243cce
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date: Sat Oct 24 18:36:24 2015 -0400
sparc64-tdep.c: Don't assign using memcpy return
This:
valbuf = memcpy (buf, valbuf, len);
causes a build failure in C++, because memcpy returns the value of
"buf" as a void *. Instead of adding a cast, we can just do the
assignment separately.
gdb/ChangeLog:
* sparc64-tdep.c (sparc64_store_arguments): Split assignment of
valbuf.
Diff:
---
gdb/ChangeLog | 5 +++++
gdb/sparc64-tdep.c | 3 ++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0a8a693..e86cbad 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
2015-10-24 Simon Marchi <simon.marchi@polymtl.ca>
+ * sparc64-tdep.c (sparc64_store_arguments): Split assignment of
+ valbuf.
+
+2015-10-24 Simon Marchi <simon.marchi@polymtl.ca>
+
* ia64-tdep.c (ia64_pseudo_register_write): Remove cast.
(ia64_push_dummy_call): Remove cast and change type of "to" to
array of gdb_byte.
diff --git a/gdb/sparc64-tdep.c b/gdb/sparc64-tdep.c
index 4c05277..a23740e 100644
--- a/gdb/sparc64-tdep.c
+++ b/gdb/sparc64-tdep.c
@@ -890,7 +890,8 @@ sparc64_store_arguments (struct regcache *regcache, int nargs,
/* Structure, Union or long double Complex arguments. */
gdb_assert (len <= 16);
memset (buf, 0, sizeof (buf));
- valbuf = memcpy (buf, valbuf, len);
+ memcpy (buf, valbuf, len);
+ valbuf = buf;
if (element % 2 && sparc64_16_byte_align_p (type))
element++;