Zengtudor/ccls
1
0
Fork 0
mirror of https://github.com/MaskRay/ccls.git synced 2025-06-10 02:02:12 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

setWarningsAsErrors(false); use SM.isInMainFile

Browse Source 
SM.isInMainFile is to work around preamble bug: spurious err_pp_unterminated_conditional with circular #include
This commit is contained in:
Fangrui Song 2018-11-10 15:05:21 -08:00 committed by Tomicyo
parent 66325538f4
commit 2e187db360
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/clang_complete.cc
View File

@ -221,7 +221,7 @@ public:
if (!L.isValid()) return;
const SourceManager &SM = Info.getSourceManager();
StringRef Filename = SM.getFilename(Info.getLocation());
bool concerned = IsConcerned(SM, Info.getLocation());
bool concerned = SM.isInMainFile(L);
auto fillDiagBase = [&](DiagBase &d) {
llvm::SmallString<64> Message;
Info.FormatDiagnostic(Message);
@ -285,6 +285,10 @@ std::unique_ptr<CompilerInstance> BuildCompilerInstance(
Clang->getDiagnostics(), Clang->getInvocation().TargetOpts));
if (!Clang->hasTarget())
return nullptr;
// Otherwise -Werror makes warnings issued as errors, which stops parsing
// prematurely when reaching the limit. This also works around the issue of
// -Wunused-parameter in interaction with SkipFunctionBodies.
Clang->getDiagnostics().setWarningsAsErrors(false);
// Construct SourceManager with UserFilesAreVolatile: true because otherwise
// RequiresNullTerminator: true may cause out-of-bounds read when a file is
// mmap'ed but is saved concurrently.