diff options
| author | Alexander Kornienko <alexfh@google.com> | 2018-11-22 16:10:18 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2018-11-22 16:10:18 +0000 |
| commit | b34b6ffa9d4aae16272cbdbf508134f98b4647d4 (patch) | |
| tree | 86a9d6ce615ed490c0ab8320e0a7ba477ea4422c /clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp | |
| parent | 840f03263072db90a20f636ab286b21523331ec6 (diff) | |
| download | llvm-b34b6ffa9d4aae16272cbdbf508134f98b4647d4.zip llvm-b34b6ffa9d4aae16272cbdbf508134f98b4647d4.tar.gz llvm-b34b6ffa9d4aae16272cbdbf508134f98b4647d4.tar.bz2 | |
[clang-tidy] Ignore template instantiations in modernize-use-using
The test I'm adding passes without the change due to the deduplication logic in
ClangTidyDiagnosticConsumer::take(). However this bug manifests in our internal
integration with clang-tidy.
I've verified the fix by locally changing LessClangTidyError to consider
replacements.
llvm-svn: 347470
Diffstat (limited to 'clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp index 5244aa6..a690e44 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp @@ -24,7 +24,8 @@ UseUsingCheck::UseUsingCheck(StringRef Name, ClangTidyContext *Context) void UseUsingCheck::registerMatchers(MatchFinder *Finder) { if (!getLangOpts().CPlusPlus11) return; - Finder->addMatcher(typedefDecl().bind("typedef"), this); + Finder->addMatcher(typedefDecl(unless(isInstantiated())).bind("typedef"), + this); } // Checks if 'typedef' keyword can be removed - we do it only if |