readelf: Format first DW_OP_GNU_implicit_pointer argument as DIE offset.

Make the printing of DW_OP_GNU_implicit_pointer consistent with other
operations that have a DIE offset as argument.

Before:
 [  6e4a]        formal_parameter
                 abstract_origin      (ref4) [  616a]
                 location             (exprloc)
                  [   0] GNU_implicit_pointer 0x6dbe, +0

After:
 [  6e4a]        formal_parameter
                 abstract_origin      (ref4) [  616a]
                 location             (exprloc)
                  [   0] GNU_implicit_pointer [  6dbe] +0

Signed-off-by: Mark Wielaard <mjw@redhat.com>

diff --git a/src/ChangeLog b/src/ChangeLog
index 5c3bf651cebbd31e926061a25d5abbc286154ea8..8373b70de9ee14f7da9b8a244794b21caed08ebe 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2013-05-06  Mark Wielaard  <mjw@redhat.com>
+
+       * readelf.c (print_ops): Format first DW_OP_GNU_implicit_pointer
+       argument as DIE offset.
+
 2013-04-24  Mark Wielaard  <mjw@redhat.com>
 
        * Makefile.am: Use AM_CPPFLAGS instead of INCLUDES.

diff --git a/src/readelf.c b/src/readelf.c
index d9527c6da3b7b149abb200dd14e4d309fdb0338f..236d0887add5ead681fbc7f63e419bb4a6ddc77c 100644 (file)
--- a/src/readelf.c
+++ b/src/readelf.c
@@ -4019,7 +4019,7 @@ print_ops (Dwfl_Module *dwflmod, Dwarf *dbg, int indent, int indentrest,
          /* Byte offset operand.  */
          get_sleb128 (sleb, data); /* XXX check overrun */
 
-         printf ("%*s[%4" PRIuMAX "] %s %#" PRIxMAX ", %+" PRId64 "\n",
+         printf ("%*s[%4" PRIuMAX "] %s [%6" PRIxMAX "] %+" PRId64 "\n",
                  indent, "", (intmax_t) offset,
                  op_name, (uintmax_t) addr, sleb);
          CONSUME (data - start);