[PATCH] D56808: Fix hexdump test

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 16 14:12:55 PST 2019


rupprecht added a comment.

test/tools/llvm-readobj/program-headers.test is doing the same thing, so you should look at that one too.

Please prefix the commit description with [llvm-readobj]



================
Comment at: test/tools/llvm-readobj/hexdump.test:2
 // Check dumping of the hexadecimal bytes of a section.
-RUN: llvm-readobj -x .text %p/../../Object/Inputs/hello-world.elf-x86-64 | FileCheck %s
+RUN: llvm-readobj -x .text %p/Inputs/trivial.obj.elf-x86-64 | FileCheck %s
 
----------------
Use test/tools/llvm-readobj/Inputs/phdrs-elf.exe-x86_64 so that the start will be non-zero, which is what the original patch was fixing.


================
Comment at: test/tools/llvm-readobj/hexdump.test:5
 CHECK: Hex dump of section '.text':
-CHECK-NEXT: {{^}}0x00400460
+CHECK-NEXT: 0x00000000 50488d3d
----------------
{{^}}0x00400610


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56808





More information about the llvm-commits mailing list