[PATCH] D52163: -S as an alias for --strip-all-gnu

Stephen Hines via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 17 02:00:00 PDT 2018


srhines added inline comments.


================
Comment at: test/tools/llvm-objcopy/strip-all-gnu.test:4
 # RUN: llvm-readobj -file-headers -sections %t2 | FileCheck %s
+# RUN: llvm-objcopy -S %t %t2
+# RUN: llvm-readobj -file-headers -sections %t2 | FileCheck %s
----------------
There's already a %t2, so you shouldn't reuse it. I also suggest you just cmp against %t2, because the results should be identical/deterministic for this flag.


Repository:
  rL LLVM

https://reviews.llvm.org/D52163





More information about the llvm-commits mailing list