Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement wasm trap handlers. #27

Merged
merged 6 commits into from
Nov 27, 2018

Conversation

sunfishcode
Copy link
Member

This adds signal handlers based on SpiderMonkey's signal-handler code.
The functionality for looking up the trap code and wasm bytecode offset
isn't yet implemented, but this is a start.

I considered rewriting this code in Rust, but decided against it for now
as C++ allows us to talk to the relevant OS APIs more directly.

Fixes #15.

This adds signal handlers based on SpiderMonkey's signal-handler code.
The functionality for looking up the trap code and wasm bytecode offset
isn't yet implemented, but this is a start.

I considered rewriting this code in Rust, but decided against it for now
as C++ allows us to talk to the relevant OS APIs more directly.

Fixes bytecodealliance#15.
@sunfishcode
Copy link
Member Author

This code is now functional on Linux at least. I haven't tested Mac or Windows yet, though the C++ code this is based on support these platforms, so if anyone tries these before I have a chance to, please report any problems!

@sunfishcode
Copy link
Member Author

Merging now as this is laying more infrastructure to build on.

@sunfishcode sunfishcode merged commit 35627cf into bytecodealliance:master Nov 27, 2018
@sunfishcode sunfishcode deleted the signalhandlers branch November 27, 2018 14:06
kubkon pushed a commit to kubkon/wasmtime that referenced this pull request Mar 11, 2020
…e#27)

and delete the submodule that was providing the path dep
pchickey pushed a commit that referenced this pull request May 20, 2021
…spacing

wasmtime-wiggle: only generate one add_{module}_to_linker func
pchickey pushed a commit to pchickey/wasmtime that referenced this pull request May 12, 2023
pchickey pushed a commit to pchickey/wasmtime that referenced this pull request May 16, 2023
dhil added a commit to dhil/wasmtime that referenced this pull request Oct 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant