[PATCH] D58027: [llvm-objcopy] Add --strip-unneeded-symbol(s)

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 03:29:56 PST 2019


mstorsjo added inline comments.


================
Comment at: test/tools/llvm-objcopy/ELF/strip-unneeded.test:29
+# RUN: llvm-objcopy --regex --strip-unneeded-symbols %t.list2.txt %t %t6
+# RUN: cmp %t2 %t6
+
----------------
All the newly added tests produce results identical to some of the existing tests, removing all symbols. Wouldn't it be useful with a test that only removes some of the unneeded symbols but not all of them, to showcase the difference?


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

https://reviews.llvm.org/D58027





More information about the llvm-commits mailing list