averello/swift-mirror
1
0
Fork 0
mirror of https://github.com/apple/swift.git synced 2025-12-14 20:36:38 +01:00
Code Issues Packages Projects Releases Wiki Activity

[ConformanceLookup] Just kidding, the compiler needs to prefer available

Browse Source 
Sendable conformances for source compatibility.

If conformance lookup always prefers the conformance from the defining module,
libraries introducing unavailable Sendable conformances can break source in
clients that declare retroactive Sendable conformances. Instead, still prefer
the available conformance, and always diagnose the client conformance as
redundant (as a warning). Then, when the retroactive conformance is removed,
the errors will surface, so the programmer has to take explicit action to
experience the source break.
This commit is contained in:
Holly Borla
2024-07-11 22:57:48 -07:00
parent 85b66d1dc2
commit b1397703a5
3 changed files with 43 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

52
lib/AST/ConformanceLookupTable.cpp
View File

@@ -69,9 +69,18 @@ void ConformanceLookupTable::ConformanceEntry::markSupersededBy(
SupersededBy = entry;
if (diagnose) {
// If an unavailable Sendable conformance is superseded by a
// retroactive one in the client, we need to record this error
// at the client decl context.
auto *dc = getDeclContext();
if (getProtocol()->isMarkerProtocol() && isFixed() &&
!entry->isFixed()) {
dc = entry->getDeclContext();
}
// Record the problem in the conformance table. We'll
// diagnose these in semantic analysis.
table.AllSupersededDiagnostics[getDeclContext()].push_back(this);
table.AllSupersededDiagnostics[dc].push_back(this);
}
}
@@ -597,21 +606,22 @@ ConformanceLookupTable::Ordering ConformanceLookupTable::compareConformances(
}
}
// Unavailable Sendable conformances cannot be replaced by available ones.
if (!lhs->getProtocol()->isMarkerProtocol()) {
// If only one of the conformances is unconditionally available on the
// current deployment target, pick that one.
//
// FIXME: Conformance lookup should really depend on source location for
// this to be 100% correct.
// FIXME: When a class and an extension with the same availability declare the
// same conformance, this silently takes the class and drops the extension.
if (lhs->getDeclContext()->isAlwaysAvailableConformanceContext() !=
rhs->getDeclContext()->isAlwaysAvailableConformanceContext()) {
return (lhs->getDeclContext()->isAlwaysAvailableConformanceContext()
? Ordering::Before
: Ordering::After);
}
// If only one of the conformances is unconditionally available on the
// current deployment target, pick that one.
//
// FIXME: Conformance lookup should really depend on source location for
// this to be 100% correct.
// FIXME: When a class and an extension with the same availability declare the
// same conformance, this silently takes the class and drops the extension.
if (lhs->getDeclContext()->isAlwaysAvailableConformanceContext() !=
rhs->getDeclContext()->isAlwaysAvailableConformanceContext()) {
// Diagnose conflicting marker protocol conformances that differ in
// un-availability.
diagnoseSuperseded = lhs->getProtocol()->isMarkerProtocol();
return (lhs->getDeclContext()->isAlwaysAvailableConformanceContext()
? Ordering::Before
: Ordering::After);
}
// If one entry is fixed and the other is not, we have our answer.
@@ -1127,9 +1137,17 @@ void ConformanceLookupTable::lookupConformances(
if (diagnostics) {
auto knownDiags = AllSupersededDiagnostics.find(dc);
if (knownDiags != AllSupersededDiagnostics.end()) {
for (const auto *entry : knownDiags->second) {
for (auto *entry : knownDiags->second) {
ConformanceEntry *supersededBy = entry->getSupersededBy();
// Diagnose the client conformance as superseded.
auto *definingModule = nominal->getParentModule();
if (entry->getDeclContext()->getParentModule() == definingModule &&
supersededBy->getDeclContext()->getParentModule() != definingModule) {
supersededBy = entry;
entry = entry->getSupersededBy();
}
diagnostics->push_back({entry->getProtocol(),
entry->getDeclaredLoc(),
entry->getKind(),