Skip to content

Commit b500f37

Browse files
committed
[Sema] Don't skip import resolution for empty file
The associated comment appears to outdated. Make sure we continue to record implicit imports for empty files.
1 parent 7f8a0e8 commit b500f37

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

lib/Sema/ImportResolution.cpp

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -276,17 +276,12 @@ class ImportResolver final : public DeclVisitor<ImportResolver> {
276276
///
277277
/// Import resolution operates on a parsed but otherwise unvalidated AST.
278278
void swift::performImportResolution(SourceFile &SF) {
279-
FrontendStatsTracer tracer(SF.getASTContext().Stats,
280-
"Import resolution");
281-
282-
// Make sure we skip adding the standard library imports if the
283-
// source file is empty.
284-
if (SF.ASTStage == SourceFile::ImportsResolved ||
285-
SF.getTopLevelDecls().empty()) {
286-
SF.ASTStage = SourceFile::ImportsResolved;
279+
// If we've already performed import resolution, bail.
280+
if (SF.ASTStage == SourceFile::ImportsResolved)
287281
return;
288-
}
289282

283+
FrontendStatsTracer tracer(SF.getASTContext().Stats,
284+
"Import resolution");
290285
ImportResolver resolver(SF);
291286

292287
// Resolve each import declaration.

0 commit comments

Comments
 (0)