-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[Concurrency] TaskExecutor ownership fixes #74000
Conversation
0837541
to
89a4eea
Compare
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
89a4eea
to
1f3e536
Compare
cdadf00
to
707551e
Compare
707551e
to
66cee59
Compare
@swift-ci please test |
@swift-ci please smoke test |
60be029
to
c310d08
Compare
@swift-ci please smoke test |
3c74d42
to
ac3d8a8
Compare
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
@swift-ci please smoke test macOS |
As only owned task executors must be released during the tasks deinit afterwards fix builtins test
creating tasks with new builtin defaulted "owned" task executor parameter.
@swift-ci please smoke test macOS |
e384026
to
8e66493
Compare
@swift-ci please smoke test |
This resolves linking issues on embedded platforms
Again not sure why embedded didn't fail here, it's supposed to run in smoke tests I thought.
Anyway, resolved and that's the last issue here 🥳 |
@swift-ci please smoke test |
We should be retaining the task executor as we take it into the Task.
Resolves rdar://128943372