[llvm] 94b8106 - [DebugInfo] Avoid repeated hash lookups (NFC) (#128301)

[via llvm-commits]

Author: Kazu Hirata
Date: 2025-02-22T02:13:02-08:00
New Revision: 94b810661a39b19f952a4e218085bc2fe67e46b3

URL: https://github.com/llvm/llvm-project/commit/94b810661a39b19f952a4e218085bc2fe67e46b3
DIFF: https://github.com/llvm/llvm-project/commit/94b810661a39b19f952a4e218085bc2fe67e46b3.diff

LOG: [DebugInfo] Avoid repeated hash lookups (NFC) (#128301)

Added: 
    

Modified: 
    llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp

Removed: 
    


################################################################################
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()); });