From 59cb47015a184862f3709a7ab084ccd086816176 Mon Sep 17 00:00:00 2001 From: "Manna, Soumi" Date: Wed, 5 Apr 2023 15:41:54 -0400 Subject: [PATCH] [NFC][clang] Fix Coverity static analyzer tool concerns about auto_causes_copy Reported by Coverity: AUTO_CAUSES_COPY Unnecessary object copies can affect performance Inside FrontendActions.cpp file, In clang::DumpModuleInfoAction::ExecuteAction(): Using the auto keyword without an & causes the copy of an object of type pair. Inside ComputeDependence.cpp file, In clang::computeDependence(clang::OverloadExpr *, bool, bool, bool): Using the auto keyword without an & causes the copy of an object of type TemplateArgumentLoc. Reviewed By: erichkeane, aaron.ballman Differential Revision: https://reviews.llvm.org/D147574 --- clang/lib/AST/ComputeDependence.cpp | 2 +- clang/lib/Frontend/FrontendActions.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/clang/lib/AST/ComputeDependence.cpp b/clang/lib/AST/ComputeDependence.cpp index eb9afbdb1c87..cd204ed62b5d 100644 --- a/clang/lib/AST/ComputeDependence.cpp +++ b/clang/lib/AST/ComputeDependence.cpp @@ -750,7 +750,7 @@ clang::computeDependence(OverloadExpr *E, bool KnownDependent, // If we have explicit template arguments, check for dependent // template arguments and whether they contain any unexpanded pack // expansions. - for (auto A : E->template_arguments()) + for (const auto &A : E->template_arguments()) Deps |= toExprDependence(A.getArgument().getDependence()); return Deps; } diff --git a/clang/lib/Frontend/FrontendActions.cpp b/clang/lib/Frontend/FrontendActions.cpp index 0349e769595d..c947772ec3e7 100644 --- a/clang/lib/Frontend/FrontendActions.cpp +++ b/clang/lib/Frontend/FrontendActions.cpp @@ -882,7 +882,7 @@ void DumpModuleInfoAction::ExecuteAction() { } // Now let's print out any modules we did not see as part of the Primary. - for (auto SM : SubModMap) { + for (const auto &SM : SubModMap) { if (!SM.second.Seen && SM.second.Mod) { Out << " " << ModuleKindName(SM.second.Kind) << " '" << SM.first << "' at index #" << SM.second.Idx