Skip to content

[DiagnosticsQoI] Sema: Use 'underlying' instead of 'concrete' in opaque result type diagnostic #63287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 30, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions include/swift/AST/DiagnosticsSema.def
Original file line number Diff line number Diff line change
Expand Up @@ -3894,8 +3894,8 @@ ERROR(unresolved_member_no_inference,none,
(DeclNameRef))
ERROR(cannot_infer_base_of_unresolved_member,none,
"cannot infer contextual base in reference to member %0", (DeclNameRef))
ERROR(cannot_infer_concrete_for_opaque_result,none,
"cannot infer concrete type for opaque result %0 from return expression",
ERROR(cannot_infer_underlying_for_opaque_result,none,
"cannot infer underlying type for opaque result %0 from return expression",
(Type))
ERROR(unresolved_nil_literal,none,
"'nil' requires a contextual type", ())
Expand Down
2 changes: 1 addition & 1 deletion lib/Sema/CSDiagnostics.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5968,7 +5968,7 @@ bool MissingGenericArgumentsFailure::diagnoseAsError() {
if (auto opaqueElt =
locator->findLast<LocatorPathElt::OpenedOpaqueArchetype>()) {
auto *opaqueDecl = opaqueElt->getDecl();
emitDiagnostic(diag::cannot_infer_concrete_for_opaque_result,
emitDiagnostic(diag::cannot_infer_underlying_for_opaque_result,
opaqueDecl->getDeclaredInterfaceType());
return true;
}
Expand Down
6 changes: 3 additions & 3 deletions test/type/opaque.swift
Original file line number Diff line number Diff line change
Expand Up @@ -581,12 +581,12 @@ do {

// https://github.com/apple/swift/issues/62787
func f62787() -> Optional<some Collection<Int>> {
return nil // expected-error{{cannot infer concrete type for opaque result 'Optional<some Collection<Int>>' from return expression}}
return nil // expected-error{{cannot infer underlying type for opaque result 'Optional<some Collection<Int>>' from return expression}}
}

func f62787_1(x: Bool) -> Optional<some Collection<Int>> {
if x {
return nil // expected-error{{cannot infer concrete type for opaque result 'Optional<some Collection<Int>>' from return expression}}
return nil // expected-error{{cannot infer underlying type for opaque result 'Optional<some Collection<Int>>' from return expression}}
}
return nil // expected-error{{cannot infer concrete type for opaque result 'Optional<some Collection<Int>>' from return expression}}
return nil // expected-error{{cannot infer underlying type for opaque result 'Optional<some Collection<Int>>' from return expression}}
}
14 changes: 7 additions & 7 deletions test/type/opaque_parameterized_existential.swift
Original file line number Diff line number Diff line change
Expand Up @@ -13,32 +13,32 @@ extension Never: P { typealias T = Never }

// I do not like them written clear
func test() -> any P<some P> { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result 'any P<some P>' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result 'any P<some P>' from return expression}}

// I do not like them nested here
func test() -> any P<[some P]> { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result 'any P<[some P]>' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result 'any P<[some P]>' from return expression}}

// I do not like them under questions
func test() -> any P<(some P)??> { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result 'any P<(some P)??>' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result 'any P<(some P)??>' from return expression}}

// I do not like meta-type intentions
func test() -> (any P<some P>).Type { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result '(any P<some P>).Type' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result '(any P<some P>).Type' from return expression}}

// I do not like them (meta)static-ly
func test() -> any P<some P>.Type { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result 'any P<some P>.Type' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result 'any P<some P>.Type' from return expression}}

// I do not like them tupled-three
func test() -> (Int, any P<some P>, Int) { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result '(Int, any P<some P>, Int)' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result '(Int, any P<some P>, Int)' from return expression}}

// I do not like them in generics
struct Wrapper<T> {}
func test() -> any P<Wrapper<some P>> { fatalError() } // expected-error {{'some' types cannot be used in constraints on existential types}}
// expected-error@-1{{cannot infer concrete type for opaque result 'any P<Wrapper<some P>>' from return expression}}
// expected-error@-1{{cannot infer underlying type for opaque result 'any P<Wrapper<some P>>' from return expression}}

// Your attempts to nest them put me in hysterics.
func test(_ x: any P<some P>) {} // expected-error {{'some' types cannot be used in constraints on existential types}}
Expand Down
4 changes: 2 additions & 2 deletions test/type/opaque_return_named.swift
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
// Tests for experimental extensions to opaque return type support.

func f0() -> <T> T { return () }
func f1() -> <T, U, V> T { () } // expected-error{{cannot infer concrete type for opaque result 'T' from return expression}}
func f1() -> <T, U, V> T { () } // expected-error{{cannot infer underlying type for opaque result 'T' from return expression}}
func f2() -> <T: Collection, U: SignedInteger> T { // expected-note{{required by opaque return type of global function 'f2()'}}
() // expected-error{{type '()' cannot conform to 'Collection'}}
// expected-note@-1{{only concrete types such as structs, enums and classes can conform to protocols}}
// expected-error@-2{{cannot infer concrete type for opaque result 'T' from return expression}}
// expected-error@-2{{cannot infer underlying type for opaque result 'T' from return expression}}
}

func f4() async -> <T> T { () }
Expand Down