[PATCH] D61672: [llvm-objcopy] Allow strip symtab in executables and DSOs

[Jake Ehrlich via Phabricator via llvm-commits]

jakehehrlich reopened this revision.
jakehehrlich added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/trunk/tools/llvm-objcopy/ELF/Object.cpp:1998
 
+static Error removeUnneededSections(Object &Obj) {
+  // We can remove an empty symbol table from non-relocatable objects.
----------------
I think this method is overly specific. Is there a reason that the user can't invoke removeSections themselves?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61672