From: Anton Kolesov <Anton.Kolesov@synopsys.com>
Date: Thu, 22 Sep 2016 10:29:43 +0000 (+0300)
Subject: arc: Fix ARI warning for printf(%p)
X-Git-Tag: users/ARM/embedded-binutils-master-2016q4~661
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=fa42dd2e8328560e65c888277ab146810c1763a8;p=thirdparty%2Fbinutils-gdb.git

arc: Fix ARI warning for printf(%p)

Replace printf ("%p") with printf ("%s", host_address_to_string ()). Printing
host addrss might make sense here because pointers can be null and this would
affect how function behaves.

This particular warning is printed only when option -Wari is passed to
contrib/ari/gdb_ari.sh

gdb/ChangeLog:

	* arc-tdep.c: Fix ARI warning for printf(%p).
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1d1908bea51..8215c41d446 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2016-09-22  Anton Kolesov  <anton.kolesov@synopsys.com>
+
+	* arc-tdep.c: Fix ARI warning for printf(%p).
+
 2016-09-21  Anton Kolesov  <anton.kolesov@synopsys.com>
 
 	* MAINTAINERS (Write After Approval): Add Anton Kolesov.
diff --git a/gdb/arc-tdep.c b/gdb/arc-tdep.c
index 6048ad6874c..7bb93adb360 100644
--- a/gdb/arc-tdep.c
+++ b/gdb/arc-tdep.c
@@ -574,8 +574,9 @@ arc_return_value (struct gdbarch *gdbarch, struct value *function,
 			  || TYPE_LENGTH (valtype) > 2 * ARC_REGISTER_SIZE);
 
   if (arc_debug)
-    debug_printf ("arc: return_value (readbuf = %p, writebuf = %p)\n",
-		  readbuf, writebuf);
+    debug_printf ("arc: return_value (readbuf = %s, writebuf = %s)\n",
+		  host_address_to_string (readbuf),
+		  host_address_to_string (writebuf));
 
   if (writebuf != NULL)
     {