Skip to content

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Oct 10, 2024

As long as they don't actually try to start any threads there is no
reason to prevent them from running/starting.

Such programs will now instead fail when they first try to create
a thread.

This is useful for programs that are built with threading support but
might also want to run in environments without SharedArrayBuffer,
e.g. when deployed without COOP/COEP.

@sbc100 sbc100 requested a review from kripken October 10, 2024 16:04
@sbc100 sbc100 force-pushed the SharedArrayBuffer branch from 2565907 to e09fde4 Compare October 10, 2024 16:06
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

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

Fair enough on both points.

@sbc100 sbc100 force-pushed the SharedArrayBuffer branch 2 times, most recently from 0d6979f to a52cc74 Compare October 10, 2024 23:14
As long as they don't actually try to start any threads there is no
reason to prevent them from running/starting.

Such programs will now instead fail when they first try to create
a thread.

This is useful for programs that are built with threading support but
might also want to run in environments without SharedArrayBuffer,
e.g. when deployed without COOP/COEP.
@sbc100 sbc100 force-pushed the SharedArrayBuffer branch from a52cc74 to 3d7f0b3 Compare October 11, 2024 05:14
@sbc100 sbc100 merged commit c55ff40 into emscripten-core:main Oct 11, 2024
1 of 2 checks passed
@sbc100 sbc100 deleted the SharedArrayBuffer branch October 11, 2024 17:17
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 11, 2024
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 11, 2024
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 11, 2024
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 11, 2024
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 14, 2024
sbc100 added a commit to sbc100/emscripten that referenced this pull request Oct 14, 2024
sbc100 added a commit that referenced this pull request Oct 15, 2024
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.

2 participants