[PATCH] D63647: [llvm-readobj/llvm-readelf] - Eliminate the elf-groups.x86_64 precompiled binary from the inputs.

Fri Jun 21 07:20:10 PDT 2019

grimar added inline comments.

================
Comment at: test/tools/llvm-readobj/elf-groups.test:47
+  Data:    ELFDATA2LSB
+  OSABI:   ELFOSABI_GNU
+  Type:    ET_REL
----------------
jhenderson wrote:
> I'm not sure the OSABI is relevant?
No. Removed.

================
Comment at: test/tools/llvm-readobj/print-section.test:1
-RUN: llvm-readobj -p .text %p/Inputs/elf-groups.x86_64 \
-RUN:     | FileCheck %s
+## Check that llvm-readobj and llvm-readelf can dump the content
+## of the section as a string.
----------------
jhenderson wrote:
> I think this test can just be deleted entirely actually. We have string-dump.test and string-dump-multi.test now.
Removed, `string-dump.test` looks good.

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

https://reviews.llvm.org/D63647