Skip to content

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