[clang-tools-extra] [clang-tidy] Avoid repeated hash lookups (NFC) (PR #127370)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 15 20:48:28 PST 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/127370
None
>From 356e99802d0d77be7fad4f9297a605334e08ffbd Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 15 Feb 2025 01:40:13 -0800
Subject: [PATCH] [clang-tidy] Avoid repeated hash lookups (NFC)
---
.../clang-tidy/bugprone/VirtualNearMissCheck.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/clang-tools-extra/clang-tidy/bugprone/VirtualNearMissCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/VirtualNearMissCheck.cpp
index 76fa2d916f0e8..509fce3a38471 100644
--- a/clang-tools-extra/clang-tidy/bugprone/VirtualNearMissCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/VirtualNearMissCheck.cpp
@@ -179,15 +179,15 @@ static bool checkOverrideByDerivedMethod(const CXXMethodDecl *BaseMD,
bool VirtualNearMissCheck::isPossibleToBeOverridden(
const CXXMethodDecl *BaseMD) {
- auto Iter = PossibleMap.find(BaseMD);
- if (Iter != PossibleMap.end())
+ auto [Iter, Inserted] = PossibleMap.try_emplace(BaseMD);
+ if (!Inserted)
return Iter->second;
bool IsPossible = !BaseMD->isImplicit() && !isa<CXXConstructorDecl>(BaseMD) &&
!isa<CXXDestructorDecl>(BaseMD) && BaseMD->isVirtual() &&
!BaseMD->isOverloadedOperator() &&
!isa<CXXConversionDecl>(BaseMD);
- PossibleMap[BaseMD] = IsPossible;
+ Iter->second = IsPossible;
return IsPossible;
}
More information about the cfe-commits
mailing list