diff options
| author | Anton Kolesov <Anton.Kolesov@synopsys.com> | 2016-09-22 13:29:43 +0300 |
|---|---|---|
| committer | Anton Kolesov <Anton.Kolesov@synopsys.com> | 2016-09-22 14:26:44 +0300 |
| commit | fa42dd2e8328560e65c888277ab146810c1763a8 (patch) | |
| tree | c9a1bcbc83ed6ed482c9835b0e991152a83ff1df | |
| parent | 870dd155d688fec15d61eb1fe7ef36bd36791dd4 (diff) | |
| download | gdb-fa42dd2e8328560e65c888277ab146810c1763a8.zip gdb-fa42dd2e8328560e65c888277ab146810c1763a8.tar.gz gdb-fa42dd2e8328560e65c888277ab146810c1763a8.tar.bz2 | |
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).
| -rw-r--r-- | gdb/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/arc-tdep.c | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1d1908b..8215c41 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 6048ad6..7bb93ad 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) { |