[opt] Change the parameter of OptTable::PrintHelp from Name to Usage and don't append...
authorFangrui Song <maskray@google.com>
Wed, 10 Oct 2018 00:15:31 +0000 (00:15 +0000)
committerFangrui Song <maskray@google.com>
Wed, 10 Oct 2018 00:15:31 +0000 (00:15 +0000)
commit88478bbc603222965b9f617acccb0d7ba409ac3b
tree75f01daa0d6fc3e928dbb7fb257aabd38266a5dc
parent7f208f02dbea6225db15dbdbf0e8d4778cc36fc5
[opt] Change the parameter of OptTable::PrintHelp from Name to Usage and don't append "[options] <inputs>"

Summary:
Before, "[options] <inputs>" is unconditionally appended to the `Name` parameter. It is more flexible to change its semantic to `Usage` and let user customize the usage line.

% llvm-objcopy
...
USAGE: llvm-objcopy <input> [ <output> ] [options] <inputs>

With this patch:

% llvm-objcopy
...
USAGE: llvm-objcopy input [output]

Reviewers: rupprecht, alexshap, jhenderson

Reviewed By: rupprecht

Subscribers: jakehehrlich, mehdi_amini, steven_wu, dexonsmith, llvm-commits

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

llvm-svn: 344097
llvm/include/llvm/Option/OptTable.h
llvm/lib/Option/OptTable.cpp
llvm/lib/ToolDrivers/llvm-dlltool/DlltoolDriver.cpp
llvm/lib/ToolDrivers/llvm-lib/LibDriver.cpp
llvm/test/tools/llvm-cvtres/help.test
llvm/test/tools/llvm-rc/helpmsg.test
llvm/tools/llvm-cvtres/llvm-cvtres.cpp
llvm/tools/llvm-mt/llvm-mt.cpp
llvm/tools/llvm-objcopy/llvm-objcopy.cpp
llvm/tools/llvm-rc/llvm-rc.cpp
llvm/unittests/Option/OptionParsingTest.cpp