[PATCH] D61377: [llvm-strip]Add --no-strip-all to disable --strip-all behaviour (including default stripping)

[James Henderson via Phabricator via llvm-commits]

jhenderson marked an inline comment as done.
jhenderson added inline comments.


================
Comment at: test/tools/llvm-objcopy/ELF/no-strip-all.test:7
+## Base case: no switches, should strip. Used as a sanity check for later test cases.
+# RUN: cp %t.o %t1.o
+# RUN: llvm-strip %t1.o
----------------
MaskRay wrote:
> Replace `cp %t.o %t1.o` with `-o %t1.o`?
No, because each of the test cases makes a copy from the pristine version before llvm-strip modifies it.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61377