Skip to content

Commit abcab4f

Browse files
[llvm-dwarfdump] Avoid repeated hash lookups (NFC) (#129991)
1 parent afe14bb commit abcab4f

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

llvm/tools/llvm-dwarfdump/Statistics.cpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -529,8 +529,9 @@ static void collectStatsRecursive(
529529
auto OffsetFn = Die.find(dwarf::DW_AT_abstract_origin);
530530
if (OffsetFn) {
531531
uint64_t OffsetOfInlineFnCopy = (*OffsetFn).getRawUValue();
532-
if (LocalAbstractOriginFnInfo.count(OffsetOfInlineFnCopy)) {
533-
AbstractOriginVars = LocalAbstractOriginFnInfo[OffsetOfInlineFnCopy];
532+
if (auto It = LocalAbstractOriginFnInfo.find(OffsetOfInlineFnCopy);
533+
It != LocalAbstractOriginFnInfo.end()) {
534+
AbstractOriginVars = It->second;
534535
AbstractOriginVarsPtr = &AbstractOriginVars;
535536
} else {
536537
// This means that the DW_AT_inline fn copy is out of order
@@ -782,9 +783,10 @@ static void collectZeroLocCovForVarsWithAbstractOrigin(
782783
// If there is no entry within LocalAbstractOriginFnInfo for the given
783784
// FnCopyRawUValue, function isn't out-of-order in DWARF. Rather, we have
784785
// CrossCU referencing.
785-
if (!LocalAbstractOriginFnInfo.count(FnCopyRawUValue))
786+
auto It = LocalAbstractOriginFnInfo.find(FnCopyRawUValue);
787+
if (It == LocalAbstractOriginFnInfo.end())
786788
continue;
787-
AbstractOriginVars = LocalAbstractOriginFnInfo[FnCopyRawUValue];
789+
AbstractOriginVars = It->second;
788790
updateVarsWithAbstractOriginLocCovInfo(FnDieWithAbstractOrigin,
789791
AbstractOriginVars);
790792

0 commit comments

Comments
 (0)