[PATCH] D38260: [llvm-objcopy] Add support for removing sections

Jake Ehrlich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 4 12:05:38 PDT 2017


jakehehrlich updated this revision to Diff 117708.
jakehehrlich added a comment.

1. Added test to make sure that segment data is kept regardless of section removal
2. Added test to make sure that .symtab can be removed without causing error
3. Added test to make sure that .shstrtab can't be removed. Once we have an option to not emit the section header table we can also add an option to allow .shstrtab to be removed in that case.


Repository:
  rL LLVM

https://reviews.llvm.org/D38260

Files:
  test/tools/llvm-objcopy/dynsym-error-remove-strtab.test
  test/tools/llvm-objcopy/reloc-error-remove-symtab.test
  test/tools/llvm-objcopy/remove-section-with-symbol.test
  test/tools/llvm-objcopy/remove-section.test
  test/tools/llvm-objcopy/remove-shstrtab-error.test
  test/tools/llvm-objcopy/remove-symtab.test
  test/tools/llvm-objcopy/segment-data-kept.test
  test/tools/llvm-objcopy/segment-shift-section-remove.test
  test/tools/llvm-objcopy/segment-test-remove-section.test
  test/tools/llvm-objcopy/symtab-error-on-remove-strtab.test
  tools/llvm-objcopy/Object.cpp
  tools/llvm-objcopy/Object.h
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38260.117708.patch
Type: text/x-patch
Size: 31161 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171004/b102e2d2/attachment.bin>


More information about the llvm-commits mailing list