[clang] [Index] Avoid repeated hash lookups (NFC) (PR #127300)

Kazu Hirata via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 14 21:34:26 PST 2025


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/127300

None

>From 99becba2d947d6f18ef6f2cfbf583b28bcd21713 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Fri, 14 Feb 2025 02:44:22 -0800
Subject: [PATCH] [Index] Avoid repeated hash lookups (NFC)

---
 clang/lib/Index/USRGeneration.cpp | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/clang/lib/Index/USRGeneration.cpp b/clang/lib/Index/USRGeneration.cpp
index 1e54b413dc59c..0a5a1bcc74865 100644
--- a/clang/lib/Index/USRGeneration.cpp
+++ b/clang/lib/Index/USRGeneration.cpp
@@ -859,16 +859,12 @@ void USRGenerator::VisitType(QualType T) {
     }
 
     // If we have already seen this (non-built-in) type, use a substitution
-    // encoding.
-    llvm::DenseMap<const Type *, unsigned>::iterator Substitution
-      = TypeSubstitutions.find(T.getTypePtr());
-    if (Substitution != TypeSubstitutions.end()) {
+    // encoding.  Otherwise, record this as a substitution.
+    auto [Substitution, Inserted] =
+        TypeSubstitutions.try_emplace(T.getTypePtr(), TypeSubstitutions.size());
+    if (!Inserted) {
       Out << 'S' << Substitution->second << '_';
       return;
-    } else {
-      // Record this as a substitution.
-      unsigned Number = TypeSubstitutions.size();
-      TypeSubstitutions[T.getTypePtr()] = Number;
     }
 
     if (const PointerType *PT = T->getAs<PointerType>()) {



More information about the cfe-commits mailing list