[llvm] fb14638 - [DebugInfo] Avoid repeated hash lookups (NFC) (#127446)

[via llvm-commits]

Author: Kazu Hirata
Date: 2025-02-17T01:32:25-08:00
New Revision: fb14638817004dc96c9401d7f704d7e5cd0ef3fc

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

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

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 932346e1b011b..513b0d312345e 100644
--- a/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
+++ b/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
@@ -65,20 +65,24 @@ LVSectionIndex LVSymbolTable::update(LVScope *Function) {
     Name = Function->getName();
   std::string SymbolName(Name);
 
-  if (SymbolName.empty() || (SymbolNames.find(SymbolName) == SymbolNames.end()))
+  if (SymbolName.empty())
+    return SectionIndex;
+
+  auto It = SymbolNames.find(SymbolName);
+  if (It == SymbolNames.end())
     return SectionIndex;
 
   // Update a recorded entry with its logical scope, only if the scope has
   // ranges. That is the case when in DWARF there are 2 DIEs connected via
   // the DW_AT_specification.
   if (Function->getHasRanges()) {
-    SymbolNames[SymbolName].Scope = Function;
-    SectionIndex = SymbolNames[SymbolName].SectionIndex;
+    It->second.Scope = Function;
+    SectionIndex = It->second.SectionIndex;
   } else {
     SectionIndex = UndefinedSectionIndex;
   }
 
-  if (SymbolNames[SymbolName].IsComdat)
+  if (It->second.IsComdat)
     Function->setIsComdat();
 
   LLVM_DEBUG({ print(dbgs()); });