diff options
author | Aaron Ballman <aaron@aaronballman.com> | 2012-07-17 23:19:16 +0000 |
---|---|---|
committer | Aaron Ballman <aaron@aaronballman.com> | 2012-07-17 23:19:16 +0000 |
commit | 8f94ac6922f46afb9dcd97ab30b4c4c3845debd4 (patch) | |
tree | 4ef4f4e2844c8420332a5bd2d93cb3b793853519 /clang/lib/Lex/PPDirectives.cpp | |
parent | 4b12ba0ce6684f2b5f3462f70d6c35de162cf31a (diff) | |
download | bcm5719-llvm-8f94ac6922f46afb9dcd97ab30b4c4c3845debd4.tar.gz bcm5719-llvm-8f94ac6922f46afb9dcd97ab30b4c4c3845debd4.zip |
Adding a fixit for includes that cannot be found with angle brackets, but can be found with quoted strings instead. Implements PR13201.
llvm-svn: 160406
Diffstat (limited to 'clang/lib/Lex/PPDirectives.cpp')
-rw-r--r-- | clang/lib/Lex/PPDirectives.cpp | 25 |
1 files changed, 22 insertions, 3 deletions
diff --git a/clang/lib/Lex/PPDirectives.cpp b/clang/lib/Lex/PPDirectives.cpp index a6b7b52624e..74b9cbc881a 100644 --- a/clang/lib/Lex/PPDirectives.cpp +++ b/clang/lib/Lex/PPDirectives.cpp @@ -1390,9 +1390,28 @@ void Preprocessor::HandleIncludeDirective(SourceLocation HashLoc, } if (File == 0) { - if (!SuppressIncludeNotFoundError) - Diag(FilenameTok, diag::err_pp_file_not_found) << Filename; - return; + if (!SuppressIncludeNotFoundError) { + // If the file could not be located and it was included via angle + // brackets, we can attempt a lookup as though it were a quoted path to + // provide the user with a possible fixit. + if (isAngled) { + File = LookupFile(Filename, false, LookupFrom, CurDir, + Callbacks ? &SearchPath : 0, + Callbacks ? &RelativePath : 0, + getLangOpts().Modules ? &SuggestedModule : 0); + if (File) { + SourceRange Range(FilenameTok.getLocation(), CharEnd); + Diag(FilenameTok, diag::err_pp_file_not_found_not_fatal) << + Filename << + FixItHint::CreateReplacement(Range, "\"" + Filename.str() + "\""); + } + } + // If the file is still not found, just go with the vanilla diagnostic + if (!File) + Diag(FilenameTok, diag::err_pp_file_not_found) << Filename; + } + if (!File) + return; } // If we are supposed to import a module rather than including the header, |