[PATCH] D61641: [llvm-objcopy] Strip file symbols with --strip-unneeded

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 7 09:06:51 PDT 2019


jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

This looks good to me, but it would be worth getting a second pair of eyes from one of the other contributors. Additionally, I'd like to have a better idea of why the decision was made to keep file symbols originally, since it was an explicit case before. I see you originally added it in https://reviews.llvm.org/rL353919.


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

https://reviews.llvm.org/D61641





More information about the llvm-commits mailing list