[PATCH] D54936: [llvm-objcopy] Hook up the -V alias to --version, output "GNU strip"

[Martin Storsjö via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL347731: [llvm-objcopy] Hook up the -V alias to --version, output "GNU strip" (authored by mstorsjo, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D54936?vs=175446&id=175630#toc

Repository:
  rL LLVM

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

https://reviews.llvm.org/D54936

Files:
  llvm/trunk/test/tools/llvm-objcopy/objcopy-version.test
  llvm/trunk/test/tools/llvm-objcopy/strip-version.test
  llvm/trunk/tools/llvm-objcopy/CopyConfig.cpp
  llvm/trunk/tools/llvm-objcopy/ObjcopyOpts.td
  llvm/trunk/tools/llvm-objcopy/StripOpts.td


Index: llvm/trunk/test/tools/llvm-objcopy/objcopy-version.test
===================================================================
--- llvm/trunk/test/tools/llvm-objcopy/objcopy-version.test
+++ llvm/trunk/test/tools/llvm-objcopy/objcopy-version.test
@@ -1,4 +1,5 @@
 # RUN: llvm-objcopy -version | FileCheck %s
 # RUN: llvm-objcopy --version | FileCheck %s
+# RUN: llvm-objcopy -V | FileCheck %s
 
 # CHECK: {{ version }}
Index: llvm/trunk/test/tools/llvm-objcopy/strip-version.test
===================================================================
--- llvm/trunk/test/tools/llvm-objcopy/strip-version.test
+++ llvm/trunk/test/tools/llvm-objcopy/strip-version.test
@@ -1,4 +1,6 @@
 # RUN: llvm-strip -version | FileCheck %s
 # RUN: llvm-strip --version | FileCheck %s
+# RUN: llvm-strip -V | FileCheck %s
 
-# CHECK: {{ version }}
+# CHECK-DAG: {{ version }}
+# CHECK-DAG: GNU strip
Index: llvm/trunk/tools/llvm-objcopy/ObjcopyOpts.td
===================================================================
--- llvm/trunk/tools/llvm-objcopy/ObjcopyOpts.td
+++ llvm/trunk/tools/llvm-objcopy/ObjcopyOpts.td
@@ -165,3 +165,4 @@
 
 def version : Flag<["-", "--"], "version">,
               HelpText<"Print the version and exit.">;
+def V : Flag<["-"], "V">, Alias<version>;
Index: llvm/trunk/tools/llvm-objcopy/CopyConfig.cpp
===================================================================
--- llvm/trunk/tools/llvm-objcopy/CopyConfig.cpp
+++ llvm/trunk/tools/llvm-objcopy/CopyConfig.cpp
@@ -226,6 +226,7 @@
   }
 
   if (InputArgs.hasArg(OBJCOPY_version)) {
+    outs() << "llvm-objcopy, compatible with GNU objcopy\n";
     cl::PrintVersionMessage();
     exit(0);
   }
@@ -383,6 +384,7 @@
   }
 
   if (InputArgs.hasArg(STRIP_version)) {
+    outs() << "llvm-strip, compatible with GNU strip\n";
     cl::PrintVersionMessage();
     exit(0);
   }
Index: llvm/trunk/tools/llvm-objcopy/StripOpts.td
===================================================================
--- llvm/trunk/tools/llvm-objcopy/StripOpts.td
+++ llvm/trunk/tools/llvm-objcopy/StripOpts.td
@@ -64,3 +64,4 @@
 
 def version : Flag<["-", "--"], "version">,
               HelpText<"Print the version and exit.">;
+def V : Flag<["-"], "V">, Alias<version>;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54936.175630.patch
Type: text/x-patch
Size: 2226 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181128/690f4697/attachment.bin>