[PATCH] D46369: [llvm-objcopy] Add --strip-symbol (-N) option

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 03:10:19 PDT 2018


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

LGTM, with nit.



================
Comment at: test/tools/llvm-objcopy/strip-group-symbol.test:26
+Symbols:
+  Local:
+    - Name:            foo
----------------
Nit: I don't think it's important, but group signature symbols are usually STB_WEAK binding.


Repository:
  rL LLVM

https://reviews.llvm.org/D46369





More information about the llvm-commits mailing list