[PATCH] D46567: [llvm-strip] Add support for -remove-section

[Jake Ehrlich via Phabricator via llvm-commits]

jakehehrlich added inline comments.


================
Comment at: test/tools/llvm-objcopy/strip-and-remove.test:1-2
+# RUN: yaml2obj %s > %t
+# RUN: llvm-objcopy -strip-all -remove-section=.text.bar %t %t
+# RUN: llvm-readobj -file-headers -sections -symbols %t | FileCheck %s
----------------
I'd prefer not regenerating the elf. You don't even need to run the FileCheck again. Principally this is an llvm-strip test and we just want to add a couple lines to make sure the equivlient llvm-objcopy produces an exactly equivalent binary. It is assumed that if the binaries are equivalent they will produce equivalent results.


Repository:
  rL LLVM

https://reviews.llvm.org/D46567