Skip to content

[region-isolation] Suppress non-Sendable region isolation errors appropriately when the type is imported by using @preconcurrency import. #73212

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

Conversation

gottesmm
Copy link
Contributor

@gottesmm gottesmm commented Apr 23, 2024

[region-isolation] Suppress non-Sendable region isolation errors appropriately when the type is imported by using @preconcurrency import.

This translates the rules for @preconcurrency import from SE-0337 into the region isolation world. Specifically if a module is compiled without strict concurrency checking and imported with @preconcurrency:

  1. All types from that module that are implicitly non-Sendable have diagnostics suppressed in swift 5 and swift 6.

  2. All types from that module that are explicitly non-Sendable emit warnings in both swift 5 and swift 6.

rdar://126804052


I still need to handle the @MainActor removal cases.

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

…opriately when the type is imported by using @preconcurrency import.

This translates the rules for @preconcurrency import from SE-0337 into the
region isolation world. Specifically if a module is compiled without strict
concurrency checking and imported with @preconcurrency:

1. All types from that module that are implicitly non-Sendable have diagnostics
suppressed in swift 5 and swift 6.

2. All types from that module that are explicitly non-Sendable emit warnings in
both swift 5 and swift 6.

rdar://126804052
@gottesmm
Copy link
Contributor Author

Had to turn off availability checking.

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

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