Skip to content

Commit 6df66e4

Browse files
kazutakahiratasivan-shani
authored andcommitted
[AST] Avoid repeated map lookups (NFC) (llvm#127369)
1 parent b74ebc6 commit 6df66e4

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

clang/lib/AST/ExternalASTMerger.cpp

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -206,16 +206,14 @@ class LazyASTImporter : public ASTImporter {
206206
<< "\n";
207207
Source<DeclContext *> FromDC(
208208
cast<DeclContext>(From)->getPrimaryContext());
209-
if (FromOrigins.count(FromDC) &&
210-
Parent.HasImporterForOrigin(*FromOrigins.at(FromDC).AST)) {
209+
if (auto It = FromOrigins.find(FromDC);
210+
It != FromOrigins.end() &&
211+
Parent.HasImporterForOrigin(*It->second.AST)) {
211212
if (LoggingEnabled)
212-
logs() << "(ExternalASTMerger*)" << (void*)&Parent
213-
<< " forced origin (DeclContext*)"
214-
<< (void*)FromOrigins.at(FromDC).DC
215-
<< ", (ASTContext*)"
216-
<< (void*)FromOrigins.at(FromDC).AST
217-
<< "\n";
218-
Parent.ForceRecordOrigin(ToDC, FromOrigins.at(FromDC));
213+
logs() << "(ExternalASTMerger*)" << (void *)&Parent
214+
<< " forced origin (DeclContext*)" << (void *)It->second.DC
215+
<< ", (ASTContext*)" << (void *)It->second.AST << "\n";
216+
Parent.ForceRecordOrigin(ToDC, It->second);
219217
} else {
220218
if (LoggingEnabled)
221219
logs() << "(ExternalASTMerger*)" << (void*)&Parent

0 commit comments

Comments
 (0)