[llvm] [DebugInfo] Avoid repeated hash lookups (NFC) (PR #128301)

[Kazu Hirata via llvm-commits]

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

None

>From 10f2727d259eac4aa6c0169cf8277192387fd684 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Fri, 21 Feb 2025 11:17:22 -0800
Subject: [PATCH] [DebugInfo] Avoid repeated hash lookups (NFC)

---
 .../LogicalView/Readers/LVBinaryReader.cpp    | 26 ++++++++-----------
 1 file changed, 11 insertions(+), 15 deletions(-)

diff --git a/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp b/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
index 513b0d312345e..ad14baa0c9269 100644
--- a/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
+++ b/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
@@ -24,18 +24,16 @@ using namespace llvm::logicalview;
 void LVSymbolTable::add(StringRef Name, LVScope *Function,
                         LVSectionIndex SectionIndex) {
   std::string SymbolName(Name);
-  if (SymbolNames.find(SymbolName) == SymbolNames.end()) {
-    SymbolNames.emplace(
-        std::piecewise_construct, std::forward_as_tuple(SymbolName),
-        std::forward_as_tuple(Function, 0, SectionIndex, false));
-  } else {
+  auto [It, Inserted] =
+      SymbolNames.try_emplace(SymbolName, Function, 0, SectionIndex, false);
+  if (!Inserted) {
     // Update a recorded entry with its logical scope and section index.
-    SymbolNames[SymbolName].Scope = Function;
+    It->second.Scope = Function;
     if (SectionIndex)
-      SymbolNames[SymbolName].SectionIndex = SectionIndex;
+      It->second.SectionIndex = SectionIndex;
   }
 
-  if (Function && SymbolNames[SymbolName].IsComdat)
+  if (Function && It->second.IsComdat)
     Function->setIsComdat();
 
   LLVM_DEBUG({ print(dbgs()); });
@@ -44,15 +42,13 @@ void LVSymbolTable::add(StringRef Name, LVScope *Function,
 void LVSymbolTable::add(StringRef Name, LVAddress Address,
                         LVSectionIndex SectionIndex, bool IsComdat) {
   std::string SymbolName(Name);
-  if (SymbolNames.find(SymbolName) == SymbolNames.end())
-    SymbolNames.emplace(
-        std::piecewise_construct, std::forward_as_tuple(SymbolName),
-        std::forward_as_tuple(nullptr, Address, SectionIndex, IsComdat));
-  else
+  auto [It, Inserted] = SymbolNames.try_emplace(SymbolName, nullptr, Address,
+                                                SectionIndex, IsComdat);
+  if (!Inserted)
     // Update a recorded symbol name with its logical scope.
-    SymbolNames[SymbolName].Address = Address;
+    It->second.Address = Address;
 
-  LVScope *Function = SymbolNames[SymbolName].Scope;
+  LVScope *Function = It->second.Scope;
   if (Function && IsComdat)
     Function->setIsComdat();
   LLVM_DEBUG({ print(dbgs()); });