[llvm] 6ad55f1 - [DebugInfo] Avoid repeated hash lookups (NFC) (#128459)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Feb 24 00:59:52 PST 2025
    
    
  
Author: Kazu Hirata
Date: 2025-02-24T00:59:49-08:00
New Revision: 6ad55f15176d10cd094c960807eac4ee2aa68a89
URL: https://github.com/llvm/llvm-project/commit/6ad55f15176d10cd094c960807eac4ee2aa68a89
DIFF: https://github.com/llvm/llvm-project/commit/6ad55f15176d10cd094c960807eac4ee2aa68a89.diff
LOG: [DebugInfo] Avoid repeated hash lookups (NFC) (#128459)
Added: 
    
Modified: 
    llvm/lib/DebugInfo/LogicalView/Readers/LVDWARFReader.cpp
Removed: 
    
################################################################################
diff  --git a/llvm/lib/DebugInfo/LogicalView/Readers/LVDWARFReader.cpp b/llvm/lib/DebugInfo/LogicalView/Readers/LVDWARFReader.cpp
index 092495e87443c..42da957233667 100644
--- a/llvm/lib/DebugInfo/LogicalView/Readers/LVDWARFReader.cpp
+++ b/llvm/lib/DebugInfo/LogicalView/Readers/LVDWARFReader.cpp
@@ -565,12 +565,8 @@ LVScope *LVDWARFReader::processOneDie(const DWARFDie &InputDIE, LVScope *Parent,
     // Insert the newly created element into the element symbol table. If the
     // element is in the list, it means there are previously created elements
     // referencing this element.
-    if (ElementTable.find(Offset) == ElementTable.end()) {
-      // No previous references to this offset.
-      ElementTable.emplace(std::piecewise_construct,
-                           std::forward_as_tuple(Offset),
-                           std::forward_as_tuple(CurrentElement));
-    } else {
+    auto [It, Inserted] = ElementTable.try_emplace(Offset, CurrentElement);
+    if (!Inserted) {
       // There are previous references to this element. We need to update the
       // element and all the references pointing to this element.
       LVElementEntry &Reference = ElementTable[Offset];
        
    
    
More information about the llvm-commits
mailing list