[PATCH] D40201: [llvm-objcopy] Add --strip-all-gnu and change --strip-all

[Jake Ehrlich via Phabricator via llvm-commits]

jakehehrlich created this revision.

GNU's --strip-all doesn't strip as aggressively as it could in general. Currently llvm-objcopy copies the exact behavoir of GNU's --strip-all. eu-strip is used as a drop in replacement for GNU strip/objcopy in many many places without issue. eu-strip removes non-allocated sections and keeps .gnu.warning* sections. Because --strip-all will likely be the most widely used stripping option we should make --strip-all as aggressive as it can safely be. Since we have evidence from eu-strip that this is a safe option we should allow it. For those that might still have an issue afterwards I've added --strip-all-gnu as an exact drop in replacement for GNU's --strip-all as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D40201

Files:
  test/tools/llvm-objcopy/strip-all-gnu.test
  test/tools/llvm-objcopy/strip-all.test
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40201.123436.patch
Type: text/x-patch
Size: 5479 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171117/f34ed2c9/attachment.bin>