Skip to content

[Concurrency] Two isolation inference changes for default isolation mode. #79044

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, 2025

Conversation

hborla
Copy link
Member

@hborla hborla commented Jan 30, 2025

  1. Non-actor initializers should participate in custom default isolation inference. Otherwise, they will not be able to touch the type's stored properties to initialize them if those properties have the default isolation inferred.
  2. All declarations inside actors and distributed actors do not participate in default isolation inference; they remain unspecified, so the existing isolation inference rules apply.

…ode.

1. Non-actor initializers should participate in custom default isolation
   inference. Otherwise, they will not be able to touch the type's stored
   properties to initialize them if those properties have the default
   isolation inferred.
2. All declarations inside actors and distributed actors do not participate
   in default isolation inference; they remain unspecified, so the existing
   isolation inference rules apply.
@hborla
Copy link
Member Author

hborla commented Jan 30, 2025

@swift-ci please smoke test

@hborla hborla merged commit b3275d9 into swiftlang:main Jan 30, 2025
3 checks passed
@hborla hborla deleted the default-actor-isolation-fixes branch January 30, 2025 16:37
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.

1 participant