[llvm] r375028 - [llvm-ar] Implement the V modifier as an alias for --version

[Jordan Rupprecht via llvm-commits]

Author: rupprecht
Date: Wed Oct 16 11:39:52 2019
New Revision: 375028

URL: http://llvm.org/viewvc/llvm-project?rev=375028&view=rev
Log:
[llvm-ar] Implement the V modifier as an alias for --version

Summary: Also update the help modifier (h) so that it works as a modifier and not just as a standalone `h`. For example, `llvm-ar h` prints the help message, but `llvm-ar xh` currently prints `unknown option h`.

Reviewers: MaskRay, gbreynoo

Subscribers: llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69007

Added:
    llvm/trunk/test/tools/llvm-ar/version.test
Modified:
    llvm/trunk/test/tools/llvm-ar/help-message.test
    llvm/trunk/tools/llvm-ar/llvm-ar.cpp

Modified: llvm/trunk/test/tools/llvm-ar/help-message.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-ar/help-message.test?rev=375028&r1=375027&r2=375028&view=diff
==============================================================================
--- llvm/trunk/test/tools/llvm-ar/help-message.test (original)
+++ llvm/trunk/test/tools/llvm-ar/help-message.test Wed Oct 16 11:39:52 2019
@@ -1,5 +1,10 @@
+## Show that the help message for llvm-ar can be printed with either the long
+## flag -help or with the h modifier.
+
 # RUN: llvm-ar h | FileCheck %s
+# RUN: llvm-ar xh | FileCheck %s
 # RUN: llvm-ar -h | FileCheck %s
+# RUN: llvm-ar -xh | FileCheck %s
 # RUN: llvm-ar -help | FileCheck %s
 # RUN: llvm-ar --help | FileCheck %s
 

Added: llvm/trunk/test/tools/llvm-ar/version.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-ar/version.test?rev=375028&view=auto
==============================================================================
--- llvm/trunk/test/tools/llvm-ar/version.test (added)
+++ llvm/trunk/test/tools/llvm-ar/version.test Wed Oct 16 11:39:52 2019
@@ -0,0 +1,11 @@
+## Show that the version for llvm-ar can be printed with either the long flag
+## -version or with the V modifier.
+
+RUN: llvm-ar V | FileCheck %s
+RUN: llvm-ar xV | FileCheck %s
+RUN: llvm-ar -V | FileCheck %s
+RUN: llvm-ar -xV | FileCheck %s
+RUN: llvm-ar -version | FileCheck %s
+RUN: llvm-ar --version | FileCheck %s
+
+CHECK: version

Modified: llvm/trunk/tools/llvm-ar/llvm-ar.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ar/llvm-ar.cpp?rev=375028&r1=375027&r2=375028&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-ar/llvm-ar.cpp (original)
+++ llvm/trunk/tools/llvm-ar/llvm-ar.cpp Wed Oct 16 11:39:52 2019
@@ -100,6 +100,7 @@ MODIFIERS:
   [b] - put [files] before [relpos] (same as [i])
   [c] - do not warn if archive had to be created
   [D] - use zero for timestamps and uids/gids (default)
+  [h] - display this help and exit
   [i] - put [files] before [relpos] (same as [b])
   [l] - ignored for compatibility
   [L] - add archive's contents
@@ -112,6 +113,7 @@ MODIFIERS:
   [u] - update only [files] newer than archive contents
   [U] - use actual timestamps and uids/gids
   [v] - be verbose about actions taken
+  [V] - display the version and exit
 )";
 
 void printHelpMessage() {
@@ -381,6 +383,12 @@ static ArchiveOperation parseCommandLine
     case 'L':
       AddLibrary = true;
       break;
+    case 'V':
+      cl::PrintVersionMessage();
+      exit(0);
+    case 'h':
+      printHelpMessage();
+      exit(0);
     default:
       fail(std::string("unknown option ") + Options[i]);
     }
@@ -1063,7 +1071,7 @@ static void runMRIScript() {
 }
 
 static bool handleGenericOption(StringRef arg) {
-  if (arg == "h" || arg.startswith("-h") || arg == "--help") {
+  if (arg == "-help" || arg == "--help") {
     printHelpMessage();
     return true;
   }