Skip to content

[wasm] Port thread-related APIs for no thread platform #4870

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

kateinoigakukun
Copy link
Member

This change ports the thread-related APIs for the no thread platform by adding no-op implementations. Note that WASI is going to support threads in the near future, so this does not guard by TARGET_OS_WASI but by !SWIFT_CORELIBS_FOUNDATION_HAS_THREADS to make it easier for future WASI + threads support.

@kateinoigakukun
Copy link
Member Author

@swift-ci test

@kateinoigakukun kateinoigakukun added the WASI WebAssembly System Interface-related changes label Feb 7, 2024
@MaxDesiatov MaxDesiatov requested a review from parkera February 7, 2024 18:34
@kateinoigakukun kateinoigakukun force-pushed the pr-c5d9ac3f9d224d05c5fd7a1445ef1f46d072610e branch from e325d29 to ba87b62 Compare February 22, 2024 04:28
@kateinoigakukun
Copy link
Member Author

@swift-ci test

This change ports the thread-related APIs for the no thread platform by
adding no-op implementations. Note that WASI is going to support threads
in the near future, so this does not guard by `TARGET_OS_WASI` but by
`!SWIFT_CORELIBS_FOUNDATION_HAS_THREADS` to make it easier for future
WASI + threads support.
@kateinoigakukun kateinoigakukun force-pushed the pr-c5d9ac3f9d224d05c5fd7a1445ef1f46d072610e branch from ba87b62 to 3744d4d Compare February 22, 2024 04:38
@kateinoigakukun
Copy link
Member Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor

@swift-ci test windows

@kateinoigakukun kateinoigakukun merged commit 806ba24 into swiftlang:main Feb 26, 2024
@kateinoigakukun kateinoigakukun deleted the pr-c5d9ac3f9d224d05c5fd7a1445ef1f46d072610e branch February 26, 2024 10:03
@compnerd
Copy link
Member

This seems to have caused a regression on macOS builds

@kateinoigakukun
Copy link
Member Author

@compnerd I apologize for that breakage. I made a fix here #4888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WASI WebAssembly System Interface-related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants