[PATCH] D65787: [llvm-strip] Support --strip-sections with the llvm-strip command.

[Jordan Rupprecht via Phabricator via llvm-commits]

rupprecht accepted this revision.
rupprecht added a comment.
This revision is now accepted and ready to land.

In D65787#1616420 <https://reviews.llvm.org/D65787#1616420>, @jhenderson wrote:

> @jakehehrlich/@rupprecht, are you aware of any reason why we SHOULDN'T have this in llvm-strip? It seems odd to have it in llvm-objcopy when the original source was eu-strip!


No, if we have the option already implemented, it should be available to both tools.



================
Comment at: llvm/test/tools/llvm-objcopy/ELF/strip-sections.test:13-15
+# RUN: cp %t %t4
+# RUN: llvm-strip --strip-sections %t4
+# RUN: cmp %t2 %t4
----------------
You can create a separate output file with `-o`

```
# RUN: llvm-strip --strip-sections %t -o %t4
# RUN: cmp %t2 %t4
```


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

https://reviews.llvm.org/D65787