Skip to content

[Concurrency] allow isolated on dynamic self types #70925

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 17, 2024

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Jan 16, 2024

isolated typechecking missed the handling of DynamicSelfType and wrongly prevented using Self with isolated.

This patch fixes the behavior and adds tests for the specific case.

Resolves #70924
Resolves rdar://121022467

@ktoso ktoso added the concurrency Feature: umbrella label for concurrency language features label Jan 16, 2024
@ktoso ktoso force-pushed the wip-dynamic-self-isolated branch from 97ced42 to 1c08aec Compare January 16, 2024 02:18
@ktoso ktoso force-pushed the wip-dynamic-self-isolated branch from 1c08aec to dde017f Compare January 16, 2024 02:19
@ktoso
Copy link
Contributor Author

ktoso commented Jan 16, 2024

@swift-ci please smoke test

@ktoso ktoso merged commit 5a3cad9 into swiftlang:main Jan 17, 2024
@ktoso ktoso deleted the wip-dynamic-self-isolated branch January 17, 2024 03:16
Catfish-Man pushed a commit to Catfish-Man/swift that referenced this pull request Jan 19, 2024
carlos4242 pushed a commit to carlos4242/swift that referenced this pull request May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concurrency Feature: umbrella label for concurrency language features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Self shorthand does not work with isolated parameters
1 participant