Skip to content

Move PROXY_TO_PTHREAD handling into native code. NFC #17153

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

Merged
merged 1 commit into from
Jun 10, 2022
Merged

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Jun 3, 2022

Fixes: #17144

@sbc100 sbc100 requested review from tlively and dschuff June 3, 2022 18:49
@sbc100 sbc100 changed the title Move PROXY_TO_PTHREAD handling all into native code. NFC Move PROXY_TO_PTHREAD handling into native code. NFC Jun 3, 2022
@sbc100 sbc100 enabled auto-merge (squash) June 3, 2022 19:27
sbc100 added a commit that referenced this pull request Jun 6, 2022
Sometimes it is useful to be able to make the runtime as alive without
also unwinding.  See test_pthread_proxying_cpp.cpp for an example of
this.

I also have use case for emscripten_runtime_keepalive_check() in #17153
sbc100 added a commit that referenced this pull request Jun 6, 2022
Sometimes it is useful to be able to make the runtime as alive without
also unwinding.  See test_pthread_proxying_cpp.cpp for an example of
this.

I also have use case for emscripten_runtime_keepalive_check() in #17153
sbc100 added a commit that referenced this pull request Jun 6, 2022
Sometimes it is useful to be able to make the runtime as alive without
also unwinding.  See `test_pthread_proxying_cpp.cpp` for an example of
this.

I also have use case for emscripten_runtime_keepalive_check() in #17153
@sbc100 sbc100 force-pushed the proxy_main branch 3 times, most recently from 6ef0530 to 0a01279 Compare June 9, 2022 20:56
@sbc100 sbc100 force-pushed the proxy_main branch 2 times, most recently from 05452d8 to 3697137 Compare June 9, 2022 23:25
@sbc100 sbc100 merged commit b86798a into main Jun 10, 2022
@sbc100 sbc100 deleted the proxy_main branch June 10, 2022 18:02
sbc100 added a commit that referenced this pull request Jun 29, 2022
This was broken in #17153 but we didn't have any tests.
sbc100 added a commit that referenced this pull request Jun 29, 2022
This was broken in #17153 but we didn't have any tests.

Fixes: #17338
sbc100 added a commit that referenced this pull request Jun 29, 2022
This was broken in #17153 but we didn't have any tests.  The fix is
to export `main` even though we don't technically need to.  We could
add a little complexity to allow metadce to remove it, but I'm not
sure its worth it.

Fixes: #17338
sbc100 added a commit that referenced this pull request Jun 29, 2022
This was broken in #17153 but we didn't have any tests.  The fix is
to export `main` even though we don't technically need to.  We could
add a little complexity to allow metadce to remove it, but I'm not
sure its worth it.

Fixes: #17338
sbc100 added a commit that referenced this pull request Jun 30, 2022
This was broken in #17153 but we didn't have any tests.  The fix is
to export `main` even though we don't technically need to.  We could
add a little complexity to allow metadce to remove it, but I'm not
sure its worth it.

Fixes: #17338
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