[PATCH] D53163: [llvm-strip] Support -s alias for --strip-all. Make both strip and objcopy case sensitive to support both -s (--strip-all) and -S (--strip-debug).
Jordan Rupprecht via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 16 16:10:10 PDT 2018
rupprecht added inline comments.
================
Comment at: test/tools/llvm-objcopy/strip-all.test:42-44
+# RUN: cp %t %t10
+# RUN: llvm-objcopy -S %t10 %t10
+# RUN: cmp %t2 %t10
----------------
jhenderson wrote:
> Nit: To reduce unnecessary changes in the diffs could you keep this as %t9, and move the llvm-strip -s case to below it, please.
Done -- I kept it here and bumped up the next case (--keep-symbol) since that seems to be a different enough test case.
Repository:
rL LLVM
https://reviews.llvm.org/D53163
More information about the llvm-commits
mailing list