[PATCH] D63393: [llvm-objdump] Remove unnecessary indentation when dumping ELF data.

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 16 22:13:40 PDT 2019


ychen marked 2 inline comments as done.
ychen added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/X86/disassemble-code-data-mix.s:17
 
 // CHECK:        b:        74 65 73 74 20 73 74 72         test str
 // CHECK-NEXT:  13:        69 6e 67 00                     ing.
----------------
MaskRay wrote:
> Assume you'll update this part when committing...
I thought I don't need to. Without this change,  the modified test should fail.

// CHECK: b: 74 65 73 74 20 73 74 72 test str
          ^
<stdin>:2:1: note: scanning from here
<stdin>:|file format ELF64-x86-64
^
<stdin>:15:2: note: possible intended match here
 b:| 74 65 73 74 20 73 74 72 test str
 ^



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63393





More information about the llvm-commits mailing list