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

[Dmitry Golovin via Phabricator via llvm-commits]

tpimh created this revision.
Herald added subscribers: llvm-commits, rupprecht, jakehehrlich.
Herald added a reviewer: alexshap.

This alias is needed for GNU binutils compatibility.  This flag is used for instance by the Linux kernel build system. I do not know if it needs a test, if it does, it's easy to add one.

This is my first patch, so please, correct me if my submission is wrong.


Repository:
  rL LLVM

https://reviews.llvm.org/D52163

Files:
  tools/llvm-objcopy/ObjcopyOpts.td


Index: tools/llvm-objcopy/ObjcopyOpts.td
===================================================================
--- tools/llvm-objcopy/ObjcopyOpts.td
+++ tools/llvm-objcopy/ObjcopyOpts.td
@@ -67,6 +67,8 @@
                 HelpText<"Remove non-allocated sections other than .gnu.warning* sections">;
 def strip_all_gnu : Flag<["-", "--"], "strip-all-gnu">,
                     HelpText<"Compaitable with GNU objcopy's --strip-all">;
+def S : JoinedOrSeparate<["-"], "S">,
+        Alias<strip_all_gnu>;
 def strip_debug : Flag<["-", "--"], "strip-debug">,
                   HelpText<"Remove all debug information">;
 def strip_dwo : Flag<["-", "--"], "strip-dwo">,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52163.165716.patch
Type: text/x-patch
Size: 664 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180917/8464d97b/attachment.bin>