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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

@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!



================
Comment at: llvm/test/tools/llvm-objcopy/ELF/strip-sections.test:15
+# RUN: llvm-strip --strip-sections %t4
+# RUN: llvm-readobj --file-headers --program-headers %t4 | FileCheck %s
+
----------------
Better would be to do a cmp of %t2 and %t4. The two outputs should be byte-for-byte identical.


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

https://reviews.llvm.org/D65787