Skip to content

Commit 846a5ad

Browse files
committed
NCGenerics: remove optimistic assertion
I thought it might be impossible to recursively lookupConformance of Copyable, unlike for Sendable. Turns out there are still situations where it can happen, that aren't invalid. This alone fixes `ClangImporter/objc_bridging_generics.swift` and will help reveal any invalid request cycles.
1 parent e6549f4 commit 846a5ad

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

lib/AST/ConformanceLookup.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,6 @@ ProtocolConformanceRef ModuleDecl::lookupConformance(Type type,
178178
ImplicitKnownProtocolConformanceRequest icvRequest{nominal, *kp};
179179
if (getASTContext().evaluator.hasActiveRequest(icvRequest) ||
180180
getASTContext().evaluator.hasActiveRequest(request)) {
181-
assert(!getInvertibleProtocolKind(*kp));
182181
return ProtocolConformanceRef::forInvalid();
183182
}
184183
}

0 commit comments

Comments
 (0)