Skip to content

Commit dbc9941

Browse files
committed
[Analyzer] Actually check for -model-path being a directory
The original patch (r348038) clearly contained a typo and checked for '-ctu-dir' twice. llvm-svn: 348125
1 parent 905a136 commit dbc9941

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

clang/lib/Frontend/CompilerInvocation.cpp

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -456,12 +456,13 @@ static void parseAnalyzerConfigs(AnalyzerOptions &AnOpts,
456456
return;
457457

458458
if (!AnOpts.CTUDir.empty() && !llvm::sys::fs::is_directory(AnOpts.CTUDir))
459-
Diags->Report(diag::err_analyzer_config_invalid_input)
460-
<< "ctu-dir" << "a filename";
459+
Diags->Report(diag::err_analyzer_config_invalid_input) << "ctu-dir"
460+
<< "a filename";
461461

462-
if (!AnOpts.CTUDir.empty() && !llvm::sys::fs::is_directory(AnOpts.CTUDir))
463-
Diags->Report(diag::err_analyzer_config_invalid_input)
464-
<< "model-path" << "a filename";
462+
if (!AnOpts.ModelPath.empty() &&
463+
!llvm::sys::fs::is_directory(AnOpts.ModelPath))
464+
Diags->Report(diag::err_analyzer_config_invalid_input) << "model-path"
465+
<< "a filename";
465466
}
466467

467468
static bool ParseMigratorArgs(MigratorOptions &Opts, ArgList &Args) {

0 commit comments

Comments
 (0)