https://github.com/smanna12 updated
https://github.com/llvm/llvm-project/pull/86808
>From 64ee510df037daeccb98a3fd9fa7e2ac5c41d279 Mon Sep 17 00:00:00 2001
From: "Manna, Soumi"
Date: Wed, 27 Mar 2024 07:41:09 -0700
Subject: [PATCH 1/2] [NFC][CLANG] Fix static analyzer bugs about unnecessary
https://github.com/cyndyishida 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
@@ -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)) {
https://github.com/cyndyishida approved this pull request.
Can you condense the title to something more like `[clang-installapi] Remove
unnecessary copy`?
A couple of nits but in general looks good. Thanks for fixing this!
https://github.com/llvm/llvm-project/pull/86808
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