Skip to content

Commit 95922d8

Browse files
[dsymutil] Avoid repeated hash lookups (NFC) (llvm#126190) (llvm#126346)
1 parent 5901bda commit 95922d8

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

llvm/tools/dsymutil/BinaryHolder.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,8 +176,8 @@ BinaryHolder::ArchiveEntry::getObjectEntry(StringRef Filename,
176176

177177
// Try the cache first.
178178
std::lock_guard<std::mutex> Lock(MemberCacheMutex);
179-
if (MemberCache.count(Key))
180-
return *MemberCache[Key];
179+
if (auto It = MemberCache.find(Key); It != MemberCache.end())
180+
return *It->second;
181181

182182
// Create a new ObjectEntry, but don't add it to the cache yet. Loading of
183183
// the archive members might fail and we don't want to lock the whole archive
@@ -228,8 +228,7 @@ BinaryHolder::ArchiveEntry::getObjectEntry(StringRef Filename,
228228
if (OE->Objects.empty())
229229
return errorCodeToError(errc::no_such_file_or_directory);
230230

231-
MemberCache[Key] = std::move(OE);
232-
return *MemberCache[Key];
231+
return *(MemberCache[Key] = std::move(OE));
233232
}
234233

235234
Expected<const BinaryHolder::ObjectEntry &>

0 commit comments

Comments
 (0)