[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