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

Add exceptional return to func_invoke in embedding doc #268

Merged
merged 11 commits into from
Apr 2, 2024

Conversation

dschuff
Copy link
Member

@dschuff dschuff commented Mar 16, 2023

This is a basic start of what we'll need to add to the embedder spec. I'm not sure if we also need a way to allow exceptions to bubble from the embedder into wasm ; I don't see any reference to what happens when the instance wants to call an import, so maybe not. Also the references aren't quite right, but I may need help to fix that

This is a basic start of what we'll need to add to the embedder spec.
I'm not sure if we also need a way to allow exceptions to bubble
from the embedder into wasm ; I don't see any reference to what
happens when the instance wants to call an import, so maybe not.
Also the references aren't quite right, but I may need help
to fix that
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
@dschuff
Copy link
Member Author

dschuff commented Apr 7, 2023

Ping.

Copy link
Member

@rossberg rossberg left a comment

Choose a reason for hiding this comment

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

Ah sorry, I forgot to click the Submit button for my comments two weeks ago!

document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
@dschuff
Copy link
Member Author

dschuff commented Mar 22, 2024

@rossberg PTAL, I rebased this on top of exnref, and also added a tag_type (as suggested by you in #269) and exception_alloc. The latter I think we will also want to use with the JS embedding's new WebAssembly.Exception.

document/core/appendix/embedding.rst Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Show resolved Hide resolved
document/core/exec/modules.rst Outdated Show resolved Hide resolved
document/core/exec/modules.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
@dschuff dschuff mentioned this pull request Mar 29, 2024
Copy link
Member

@rossberg rossberg left a comment

Choose a reason for hiding this comment

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

LGTM modulo comments

document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
document/core/appendix/embedding.rst Outdated Show resolved Hide resolved
@dschuff dschuff merged commit 153ca9a into main Apr 2, 2024
5 checks passed
@aheejin aheejin deleted the embedding-invoke branch April 9, 2024 13:13
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.

None yet

3 participants