[PATCH] D46896: [llvm-objcopy] Add --strip-unneeded option
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 24 07:39:32 PDT 2018
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
More information about the llvm-commits
mailing list