Skip to content
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

Improve some pthreads stub functions, batch 0 #519

Merged
merged 3 commits into from
Aug 2, 2024

Conversation

ArcaneNibble
Copy link
Contributor

This is one part of breaking up #518 into smaller PRs. This should be (IMO) the least-controversial batch of commits

The Open Group Base Specifications does not list this as an optional function

Return EINVAL for anything that isn't 0
The Open Group Base Specifications list init and destroy as required

{get|set}pshared aren't required nor useful as WASI doesn't have subprocesses
nor the ability to remap multiple virtual memory pages to the same contents.
However, for consistency with other types, provide them as stubs.
@ArcaneNibble
Copy link
Contributor Author

I don't have the ability to merge this, do we need more reviewers or does somebody just need to click the button?

@sbc100 sbc100 merged commit 5d3c5e9 into WebAssembly:main Aug 2, 2024
8 checks passed
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.

3 participants