replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Clean up timing command line option. 

Use getNumOccurrences instead of usina a default constant string.

llvm-svn: 191308
This commit is contained in:
Ariel J. Bernal 2013-09-24 15:41:30 +00:00
parent 35729bb1f8
commit e5ba87b6bb
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
clang-tools-extra/clang-modernize/tool/ClangModernize.cpp
View File

@ -93,11 +93,10 @@ static cl::opt<bool>
SummaryMode("summary", cl::desc("Print transform summary"),
cl::init(false));
const char NoTiming[] = "no_timing";
static cl::opt<std::string> TimingDirectoryName(
"perf", cl::desc("Capture performance data and output to specified "
"directory. Default: ./migrate_perf"),
cl::init(NoTiming), cl::ValueOptional, cl::value_desc("directory name"));
cl::ValueOptional, cl::value_desc("directory name"));
static cl::opt<std::string>
IncludePaths("include",
@ -384,9 +383,8 @@ int main(int argc, const char **argv) {
return 1;
}
// Since ExecutionTimeDirectoryName could be an empty string we compare
// against the default value when the command line option is not specified.
GlobalOptions.EnableTiming = (TimingDirectoryName != NoTiming);
// Enable timming.
GlobalOptions.EnableTiming = TimingDirectoryName.getNumOccurrences() > 0;
// Check the reformatting style option
bool CmdSwitchError = false;