[llvm] r340229 - [llvm-strip] Allow only one input

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 16:01:57 PDT 2018


Author: maskray
Date: Mon Aug 20 16:01:57 2018
New Revision: 340229

URL: http://llvm.org/viewvc/llvm-project?rev=340229&view=rev
Log:
[llvm-strip] Allow only one input

Summary: Before, llvm-strip accepted a second argument but it would just be ignored.

Reviewers: alexshap, jhenderson, paulsemel

Reviewed By: alexshap

Subscribers: jakehehrlich, rupprecht, llvm-commits

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

Modified:
    llvm/trunk/tools/llvm-objcopy/llvm-objcopy.cpp

Modified: llvm/trunk/tools/llvm-objcopy/llvm-objcopy.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-objcopy/llvm-objcopy.cpp?rev=340229&r1=340228&r2=340229&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-objcopy/llvm-objcopy.cpp (original)
+++ llvm/trunk/tools/llvm-objcopy/llvm-objcopy.cpp Mon Aug 20 16:01:57 2018
@@ -930,12 +930,12 @@ static CopyConfig parseStripOptions(Arra
       T.ParseArgs(ArgsArr, MissingArgumentIndex, MissingArgumentCount);
 
   if (InputArgs.size() == 0) {
-    T.PrintHelp(errs(), "llvm-strip <input> [ <output> ]", "strip tool");
+    T.PrintHelp(errs(), "llvm-strip", "strip tool");
     exit(1);
   }
 
   if (InputArgs.hasArg(STRIP_help)) {
-    T.PrintHelp(outs(), "llvm-strip <input> [ <output> ]", "strip tool");
+    T.PrintHelp(outs(), "llvm-strip", "strip tool");
     exit(0);
   }
 
@@ -948,7 +948,7 @@ static CopyConfig parseStripOptions(Arra
   if (Positional.empty())
     error("No input file specified");
 
-  if (Positional.size() > 2)
+  if (Positional.size() > 1)
     error("Support for multiple input files is not implemented yet");
 
   CopyConfig Config;




More information about the llvm-commits mailing list