Skip to content

Commit f7b1f71

Browse files
authored
Merge pull request #61243 from tshortli/solver-ignore-decls-in-unavailable-extensions
[ConstraintSystem] Extend availability check to cover unavailable extensions
2 parents c16eadb + 6de6de2 commit f7b1f71

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

lib/Sema/ConstraintSystem.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6513,7 +6513,7 @@ void ConstraintSystem::diagnoseFailureFor(SolutionApplicationTarget target) {
65136513
bool ConstraintSystem::isDeclUnavailable(const Decl *D,
65146514
ConstraintLocator *locator) const {
65156515
// First check whether this declaration is universally unavailable.
6516-
if (D->getAttrs().isUnavailable(getASTContext()))
6516+
if (AvailableAttr::isUnavailable(D))
65176517
return true;
65186518

65196519
return TypeChecker::isDeclarationUnavailable(D, DC, [&] {

test/Constraints/availability.swift

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,25 @@ func unavailableFunction(_ x: Int) -> Bool { true } // expected-note {{'unavaila
4343
func f_55700(_ arr: [Int]) {
4444
for x in arr where unavailableFunction(x) {} // expected-error {{'unavailableFunction' is unavailable}}
4545
}
46+
47+
// rdar://92364955 - ambiguity with member declared in unavailable extension
48+
struct WithUnavailableExt {
49+
}
50+
51+
@available(*, unavailable)
52+
extension WithUnavailableExt {
53+
static var foo: WithUnavailableExt = WithUnavailableExt()
54+
}
55+
56+
func test_no_ambiguity_with_unavailable_ext() {
57+
struct A {
58+
static var foo: A = A()
59+
}
60+
61+
struct Test {
62+
init(_: A) {}
63+
init(_: WithUnavailableExt) {}
64+
}
65+
66+
_ = Test(.foo) // Ok `A.foo` since `foo` from `WithUnavailableExt` is unavailable
67+
}

0 commit comments

Comments
 (0)