[PATCH] D56726: Include section start address when dumping hexadecimal output of a section, -x option.q

[Sid Manning via Phabricator via llvm-commits]

sidneym updated this revision to Diff 181803.
sidneym added a comment.

Include full context


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56726/new/

https://reviews.llvm.org/D56726

Files:
  test/tools/llvm-readobj/hexdump.test
  tools/llvm-readobj/ObjDumper.cpp


Index: tools/llvm-readobj/ObjDumper.cpp
===================================================================
--- tools/llvm-readobj/ObjDumper.cpp
+++ tools/llvm-readobj/ObjDumper.cpp
@@ -117,7 +117,8 @@
     uint8_t i;
     uint8_t k;
 
-    W.startLine() << format_hex(SecPtr - SecContent, 10);
+    W.startLine() << format_hex(Section.getAddress() + (SecPtr - SecContent),
+                                10);
     W.startLine() << ' ';
     for (i = 0; TmpSecPtr < SecEnd && i < 4; ++i) {
       for (k = 0; TmpSecPtr < SecEnd && k < 4; k++, TmpSecPtr++) {
Index: test/tools/llvm-readobj/hexdump.test
===================================================================
--- /dev/null
+++ test/tools/llvm-readobj/hexdump.test
@@ -0,0 +1,6 @@
+// Check dumping of the hexadecimal bytes of a section
+RUN: llvm-readobj -x .text %p/../../Object/Inputs/hello-world.elf-x86-64 | FileCheck %s
+
+CHECK: Hex dump of section '.text':
+CHECK-NEXT: 0x00400460
+


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56726.181803.patch
Type: text/x-patch
Size: 953 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190115/b2236036/attachment.bin>