diff options
| -rw-r--r-- | clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp b/clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp index cde30a621bc..a34f7076ea7 100644 --- a/clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp +++ b/clang-tools-extra/clang-tidy/performance/UnnecessaryValueParamCheck.cpp @@ -68,15 +68,14 @@ UnnecessaryValueParamCheck::UnnecessaryValueParamCheck( void UnnecessaryValueParamCheck::registerMatchers(MatchFinder *Finder) { const auto ExpensiveValueParamDecl = - parmVarDecl(hasType(hasCanonicalType(allOf(matchers::isExpensiveToCopy(), - unless(referenceType())))), + parmVarDecl(hasType(hasCanonicalType(allOf( + unless(referenceType()), matchers::isExpensiveToCopy()))), decl().bind("param")); Finder->addMatcher( - functionDecl(hasBody(stmt()), isDefinition(), + functionDecl(hasBody(stmt()), isDefinition(), unless(isImplicit()), unless(cxxMethodDecl(anyOf(isOverride(), isFinal()))), - unless(anyOf(isInstantiated(), isImplicit())), has(typeLoc(forEach(ExpensiveValueParamDecl))), - decl().bind("functionDecl")), + unless(isInstantiated()), decl().bind("functionDecl")), this); } |
