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

[James Henderson via Phabricator via llvm-commits]

jhenderson accepted this revision.
jhenderson added a comment.

I think this makes sense to me. LGTM, with one nit, but obviously please wait for @jakehehrlich to give his opinion.



================
Comment at: tools/llvm-objcopy/Object.cpp:189
   Sym.Index = Symbols.size();
+  Sym.Referenced = false;
   Symbols.emplace_back(llvm::make_unique<Symbol>(Sym));
----------------
This could probably be initialised in the class inline.


Repository:
  rL LLVM

https://reviews.llvm.org/D46896