[clang] [AST] Avoid repeated hash lookups (NFC) (PR #126400)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 8 21:24:58 PST 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/126400
None
>From b070c73dc8c47453ff3cc803d55ebdb79e38172d Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 8 Feb 2025 13:34:44 -0800
Subject: [PATCH] [AST] Avoid repeated hash lookups (NFC)
---
clang/lib/AST/VTableBuilder.cpp | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/clang/lib/AST/VTableBuilder.cpp b/clang/lib/AST/VTableBuilder.cpp
index fa3055dd1206f41..19d76df99dbe310 100644
--- a/clang/lib/AST/VTableBuilder.cpp
+++ b/clang/lib/AST/VTableBuilder.cpp
@@ -1169,12 +1169,13 @@ void ItaniumVTableBuilder::ComputeThisAdjustments() {
//
// Do not set ThunkInfo::Method if Idx is already in VTableThunks. This
// can happen when covariant return adjustment is required too.
- if (!VTableThunks.count(Idx)) {
+ auto [It, Inserted] = VTableThunks.try_emplace(Idx);
+ if (Inserted) {
const CXXMethodDecl *Method = VTables.findOriginalMethodInMap(MD);
- VTableThunks[Idx].Method = Method;
- VTableThunks[Idx].ThisType = Method->getThisType().getTypePtr();
+ It->second.Method = Method;
+ It->second.ThisType = Method->getThisType().getTypePtr();
}
- VTableThunks[Idx].This = ThisAdjustment;
+ It->second.This = ThisAdjustment;
};
SetThisAdjustmentThunk(VTableIndex);
@@ -1653,8 +1654,9 @@ void ItaniumVTableBuilder::AddMethods(
// findOriginalMethod to find the method that created the entry if the
// method in the entry requires adjustment.
if (!ReturnAdjustment.isEmpty()) {
- VTableThunks[Components.size()].Method = MD;
- VTableThunks[Components.size()].ThisType = MD->getThisType().getTypePtr();
+ auto &VTT = VTableThunks[Components.size()];
+ VTT.Method = MD;
+ VTT.ThisType = MD->getThisType().getTypePtr();
}
AddMethod(Overrider.Method, ReturnAdjustment);
More information about the cfe-commits
mailing list