Skip to content

Concurrency: Guard use of withoutActuallyEscaping() in inlinable code #72259

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

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented Mar 12, 2024

Since withoutActuallyEscaping() has adopted typed throws, it's no longer visible to older compilers that do not support typed throws. We need to guard use of the function in inlinable code to make sure the textual interface of _Concurrency remains buildable with older compilers.

Resolves rdar://124352900

@tshortli tshortli requested a review from DougGregor March 12, 2024 17:26
@tshortli tshortli requested a review from ktoso as a code owner March 12, 2024 17:26
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli tshortli force-pushed the typed-throws-without-actually-escaping branch from 70a780d to 40c1a3c Compare March 12, 2024 17:27
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

Since `withoutActuallyEscaping()` has adopted typed throws, it's no longer
visible to older compilers that do not support typed throws. We need to guard
use of the function in inlinable code to make sure the textual interface of
`_Concurrency` remains buildable with older compilers.

Resolves rdar://124352900
@tshortli tshortli force-pushed the typed-throws-without-actually-escaping branch from 40c1a3c to 65c0ef8 Compare March 12, 2024 17:35
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli tshortli enabled auto-merge March 12, 2024 18:08
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test Linux

@tshortli tshortli merged commit 32cd086 into swiftlang:main Mar 12, 2024
@tshortli tshortli deleted the typed-throws-without-actually-escaping branch March 12, 2024 23:14
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