Skip to content

Commit 66503ea

Browse files
author
David Ungar
committed
Fix test typo & remove too-strict assertion.
1 parent fbaa9ab commit 66503ea

File tree

3 files changed

+2
-20
lines changed

3 files changed

+2
-20
lines changed

include/swift/AST/TypeCheckRequests.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,6 @@ class DefaultTypeRequest
362362
TypeChecker &getTypeChecker() const;
363363
SourceFile *getSourceFile() const;
364364
Type &getCache() const;
365-
bool isDependencyMissing(Type result) const;
366365
};
367366

368367
/// The zone number for the type checker.

lib/AST/TypeCheckRequests.cpp

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -463,24 +463,7 @@ Type &DefaultTypeRequest::getCache() const {
463463

464464
Optional<Type> DefaultTypeRequest::getCachedResult() const {
465465
auto const &cachedType = getCache();
466-
if (!cachedType)
467-
return None;
468-
assert(!isDependencyMissing(cachedType) &&
469-
"Since the cache is now cached by SourceFile, the dependency should "
470-
"have been recorded when it was looked up.");
471-
return cachedType;
472-
}
473-
474-
bool DefaultTypeRequest::isDependencyMissing(Type result) const {
475-
if (auto *SF = getSourceFile())
476-
if (auto *tracker = SF->getReferencedNameTracker()) {
477-
Identifier name = getDeclContext()->getASTContext().getIdentifier(
478-
getTypeName(getKnownProtocolKind()));
479-
DeclBaseName bn = DeclBaseName(name);
480-
return tracker->getTopLevelNames().find(bn) ==
481-
tracker->getTopLevelNames().end();
482-
}
483-
return false;
466+
return cachedType ? Optional<Type>(cachedType) : None;
484467
}
485468

486469
void DefaultTypeRequest::cacheResult(Type value) const { getCache() = value; }

test/NameBinding/reference-dependencies-consistency.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@
1212
//
1313
// RUN: %target-swift-frontend -typecheck -primary-file %t/1.swift -primary-file %t/2.swift -emit-reference-dependencies-path %t/1.swiftdeps -emit-reference-dependencies-path %t/2.swiftdeps
1414
//
15-
// RUN cmp -s %t/1.swiftdeps %t/2.swiftdeps
15+
// RUN: cmp -s %t/1.swiftdeps %t/2.swiftdeps

0 commit comments

Comments
 (0)