[PATCH] D46896: [llvm-objcopy] Add --strip-unneeded option

[Jake Ehrlich via Phabricator via llvm-commits]

jakehehrlich added a comment.

As always the right solution winds up 10000 times simpler than anything I considered in the interim. This looks good to me after fixing Alex's and my comments.



================
Comment at: tools/llvm-objcopy/Object.cpp:950
 
+void Object::markSymbols() {
+  for (const SecPtr &Sec : Sections)
----------------
This can be done externally via a loop over `sections()`. I'd like to keep the exposed methods as simple as possible.


Repository:
  rL LLVM

https://reviews.llvm.org/D46896