Skip to content

[6.0] Downgrade the error about #isolation in @_unsafeInheritExecutor when the concurrency library is old #75348

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

Conversation

DougGregor
Copy link
Member

  • Explanation: The introduction of the error concerning use of #isolation within an @_unsafeInheritExecutor function was carefully staged between the concurrency library and compiler to avoid breaking source compatibility. However, it is a source compatibility break when dealing certain revisions of the concurrency library that adopted #isolation prior to the compiler error. Detect when the _Concurrency library doesn't have the necessary workarounds in it and downgrade the error to a warning.
  • Scope: Narrow change to downgrade a specific error to a warning in @_unsafeInheritExecutor functions.
  • Issues: rdar://132023573
  • Original PRs: Downgrade the error about #isolation in @_unsafeInheritExecutor when the concurrency library is old #75347
  • Risk: Low, very narrow change.
  • Testing: Regression tests, plus testing of the workaround with the One Version of the concurrency library that's causing the issue.

…hen the concurrency library is old

The introduction of the error concerning use of `#isolation` within an
`@_unsafeInheritExecutor` function was carefully staged between the
concurrency library and compiler to avoid breaking source
compatibility. However, it is a source compatibility break when
dealing certain revisions of the concurrency library that adopted
`#isolation` prior to the compiler error. Detect when the _Concurrency
library doesn't have the necessary workarounds in it and downgrade the
error to a warning.

Fixes rdar://132023573.
@DougGregor DougGregor requested a review from a team as a code owner July 18, 2024 18:46
@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@swift-ci please test macOS

@DougGregor DougGregor enabled auto-merge July 18, 2024 23:06
@DougGregor DougGregor merged commit e177316 into swiftlang:release/6.0 Jul 19, 2024
5 checks passed
@DougGregor DougGregor deleted the isolation-in-unsafe-inherit-downgrade-old-concurrency-lib-6.0 branch July 19, 2024 05:19
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.

3 participants