llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Amila Senadheera (Amila-Rukshan) <details> <summary>Changes</summary> Fix the cppcheck identified performance issue: https://github.com/llvm/llvm-project/issues/87248 I ran the following to check the possible perf issue in `clang/lib/ASTMatchers/Dynamic` directory: ``` cppcheck --enable=performance --language=c++ . ``` --- Full diff: https://github.com/llvm/llvm-project/pull/87273.diff 1 Files Affected: - (modified) clang/lib/ASTMatchers/Dynamic/Marshallers.h (+1-1) ``````````diff diff --git a/clang/lib/ASTMatchers/Dynamic/Marshallers.h b/clang/lib/ASTMatchers/Dynamic/Marshallers.h index c76ddf17b719d4..fc0771d7496df2 100644 --- a/clang/lib/ASTMatchers/Dynamic/Marshallers.h +++ b/clang/lib/ASTMatchers/Dynamic/Marshallers.h @@ -936,7 +936,7 @@ class MapAnyOfMatcherDescriptor : public MatcherDescriptor { public: MapAnyOfMatcherDescriptor(ASTNodeKind CladeNodeKind, - std::vector<ASTNodeKind> NodeKinds) + const std::vector<ASTNodeKind>& NodeKinds) : CladeNodeKind(CladeNodeKind), NodeKinds(NodeKinds) {} VariantMatcher create(SourceRange NameRange, ArrayRef<ParserValue> Args, `````````` </details> https://github.com/llvm/llvm-project/pull/87273 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
