nicolas/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Change -analyzer-config to use '=' as the key-value separator, and only

Browse Source 
support the '-analyzer-config key=val' variant.

llvm-svn: 162891
This commit is contained in:
Ted Kremenek
2012-08-30 05:49:16 +00:00
parent 723b4f02a7
commit 53f3e7dba3
3 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/Frontend/CompilerInvocation.cpp
View File

@@ -1161,20 +1161,20 @@ static bool ParseAnalyzerArgs(AnalyzerOptions &Opts, ArgList &Args,
const Arg *A = *it;
A->claim();
// We can have a list of comma separated config names, e.g:
// '-analyzer-config=key1:val1,key2:val2'
// '-analyzer-config key1=val1,key2=val2'
StringRef configList = A->getValue(Args);
SmallVector<StringRef, 4> configVals;
configList.split(configVals, ",");
for (unsigned i = 0, e = configVals.size(); i != e; ++i) {
StringRef key, val;
llvm::tie(key, val) = configVals[i].split(":");
llvm::tie(key, val) = configVals[i].split("=");
if (val.empty()) {
Diags.Report(SourceLocation(),
diag::err_analyzer_config_no_value) << configVals[i];
Success = false;
break;
}
if (val.find(':') != StringRef::npos) {
if (val.find('=') != StringRef::npos) {
Diags.Report(SourceLocation(),
diag::err_analyzer_config_multiple_values)
<< configVals[i];