[PATCH] D61727: [llvm-objcopy] Add additional testing for various cases

[James Henderson via Phabricator via llvm-commits]

jhenderson updated this revision to Diff 198813.
jhenderson marked 2 inline comments as done.
jhenderson added a comment.

Address review comments:

- Use extra indentation
- Use r+b for python binary editing


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61727

Files:
  test/tools/llvm-objcopy/ELF/common-suffix-string-tables.test
  test/tools/llvm-objcopy/ELF/invalid-e_phoff.test
  test/tools/llvm-objcopy/ELF/invalid-e_shoff.test
  test/tools/llvm-objcopy/ELF/linked-section.test
  test/tools/llvm-objcopy/ELF/overlapping-sections-in-segments.test
  test/tools/llvm-objcopy/ELF/overlapping-sections.test
  test/tools/llvm-objcopy/ELF/rename-section-duplicate-names.test
  test/tools/llvm-objcopy/ELF/rename-section-empty-string.test
  test/tools/llvm-objcopy/ELF/shared-strtab-shstrtab.s
  test/tools/llvm-objcopy/ELF/strip-sections.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61727.198813.patch
Type: text/x-patch
Size: 19005 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190509/c2cd5afd/attachment.bin>