[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