Skip to content

[Property Wrappers] Use the outermost wrapper attribute type as the contextual type for property wrapper initialization #31002

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
Apr 14, 2020

Conversation

hborla
Copy link
Member

@hborla hborla commented Apr 14, 2020

Otherwise, the locator path for the property wrapper backing initializer call could be rooted in a contextual type path element with no contextual type for that expression in the constraint system, which breaks assumptions in diagnostics code.

Resolves: rdar://problem/60504098

@hborla hborla requested a review from xedin April 14, 2020 00:18
@hborla
Copy link
Member Author

hborla commented Apr 14, 2020

@swift-ci please smoke test

@hborla hborla force-pushed the property-wrapper-contextual-type branch from 1b64406 to 6144ec3 Compare April 14, 2020 00:23
@hborla
Copy link
Member Author

hborla commented Apr 14, 2020

@swift-ci please smoke test

@hborla hborla force-pushed the property-wrapper-contextual-type branch from 6144ec3 to 4206587 Compare April 14, 2020 00:32
@hborla
Copy link
Member Author

hborla commented Apr 14, 2020

@swift-ci please smoke test

contextual type for property wrapper initialization
@hborla hborla force-pushed the property-wrapper-contextual-type branch from 4206587 to df7735e Compare April 14, 2020 01:18
@hborla
Copy link
Member Author

hborla commented Apr 14, 2020

@swift-ci please smoke test

@hborla hborla merged commit 48b6bcd into swiftlang:master Apr 14, 2020
@hborla hborla deleted the property-wrapper-contextual-type branch April 14, 2020 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants