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

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 04:55:56 PDT 2019


jhenderson updated this revision to Diff 199003.
jhenderson marked 3 inline comments as done.
jhenderson added a comment.

Address review comments:

- Add extra spacing.
- Rename tail merging test/update comments.
- Add extra symbol and section to tail merging test.
- Remove unnecessary switches from tail merging test.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61727

Files:
  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
  test/tools/llvm-objcopy/ELF/tail-merged-string-tables.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61727.199003.patch
Type: text/x-patch
Size: 21937 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190510/4a355ca7/attachment.bin>


More information about the llvm-commits mailing list