[clang] [clang-installapi] Remove unnecessary copy (PR #86808)

2024-03-27 Thread via cfe-commits 

https://github.com/smanna12 closed 
https://github.com/llvm/llvm-project/pull/86808
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-installapi] Remove unnecessary copy (PR #86808)

2024-03-27 Thread via cfe-commits 


@@ -255,7 +255,7 @@ bool InstallAPIVisitor::VisitFunctionDecl(const 
FunctionDecl *D) {
   return true;
 
 // Skip methods in CXX RecordDecls.
-for (auto P : D->getASTContext().getParents(*M)) {
+for (const auto  : D->getASTContext().getParents(*M)) {

smanna12 wrote:

Thanks you @cyndyishida for the reviews. I have updated patch to be explicit 
about the type and the title. 

https://github.com/llvm/llvm-project/pull/86808
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-installapi] Remove unnecessary copy (PR #86808)

2024-03-27 Thread via cfe-commits 

https://github.com/smanna12 edited 
https://github.com/llvm/llvm-project/pull/86808
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits