diff options
| author | Haojian Wu <hokein@google.com> | 2016-06-27 08:04:01 +0000 |
|---|---|---|
| committer | Haojian Wu <hokein@google.com> | 2016-06-27 08:04:01 +0000 |
| commit | e5d277923a80017dec16f0d65a793a59b80bc4c7 (patch) | |
| tree | 11ada1fae73c30d74426ec00b261b6ff561e97c1 /clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp | |
| parent | a45da385f8501a87be53ce5fda0d71d741217e76 (diff) | |
| download | bcm5719-llvm-e5d277923a80017dec16f0d65a793a59b80bc4c7.tar.gz bcm5719-llvm-e5d277923a80017dec16f0d65a793a59b80bc4c7.zip | |
[clang-tidy] Don't run misc-definitions-in-headers check in failing TUs.
Reviewers: alexfh
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D21470
llvm-svn: 273849
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp b/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp index e91cc9573f7..25fc09288b0 100644 --- a/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp @@ -72,6 +72,10 @@ void DefinitionsInHeadersCheck::registerMatchers(MatchFinder *Finder) { } void DefinitionsInHeadersCheck::check(const MatchFinder::MatchResult &Result) { + // Don't run the check in failing TUs. + if (Result.Context->getDiagnostics().hasErrorOccurred()) + return; + // C++ [basic.def.odr] p6: // There can be more than one definition of a class type, enumeration type, // inline function with external linkage, class template, non-static function |
