Skip to content

Enable MainActor in embedded concurrency, add ExecutorImpl.cpp #82331

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Enable MainActor in embedded concurrency, add ExecutorImpl.cpp #82331

wants to merge 1 commit into from
+19 −16

Conversation

MaxDesiatov
Copy link
Contributor

ExecutorImpl.cpp should be moved from SWIFT_RUNTIME_CONCURRENCY_NONEMBEDDED_C_SOURCES to SWIFT_RUNTIME_CONCURRENCY_C_SOURCES. This way we can also include ExecutorImpl.swift and use PlatformExecutorCooperative.swift in embedded concurrency for WASI.

@MaxDesiatov
Enable MainActor in embedded concurrency, add ExecutorImpl.cpp
1bd782a 
`ExecutorImpl.cpp` should be moved from `SWIFT_RUNTIME_CONCURRENCY_NONEMBEDDED_C_SOURCES` to `SWIFT_RUNTIME_CONCURRENCY_C_SOURCES`. This way we can also include `ExecutorImpl.swift` and use `PlatformExecutorCooperative.swift` in embedded concurrency for WASI.
@MaxDesiatov MaxDesiatov requested a review from ktoso as a code owner June 18, 2025 10:31
@MaxDesiatov MaxDesiatov added Concurrencу Area → standard library: The `Concurrency` module under the standard library umbrella concurrency runtime Feature: umbrella label for concurrency runtime features embedded Embedded Swift WebAssembly Platform: WebAssembly WASI WebAssembly System Interface support labels Jun 18, 2025
ktoso
ktoso approved these changes Jun 18, 2025
View reviewed changes
Copy link
Contributor

@ktoso ktoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok I think

@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ktoso ktoso ktoso approved these changes

@phausler phausler Awaiting requested review from phausler

@kubamracek kubamracek Awaiting requested review from kubamracek

@al45tair al45tair Awaiting requested review from al45tair

@rauhul rauhul Awaiting requested review from rauhul

@kateinoigakukun kateinoigakukun Awaiting requested review from kateinoigakukun

Assignees
No one assigned
Labels
Concurrencу Area → standard library: The `Concurrency` module under the standard library umbrella concurrency runtime Feature: umbrella label for concurrency runtime features embedded Embedded Swift WASI WebAssembly System Interface support WebAssembly Platform: WebAssembly
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MaxDesiatov @ktoso