Stream: git-wasmtime

Topic: wasmtime / PR #11495 Support exceptions with Pulley


view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:36):

alexcrichton opened PR #11495 from alexcrichton:pulley-exceptions to bytecodealliance:main:

This implements Chris's suggestion to avoid special-casing the raise libcall and instead let it fully execute for Pulley. The switch of whether to do a native longjmp or a Pulley longjmp is done based on the interpreter state of Wasmtime's configured executor.

Closes #11486

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:36):

alexcrichton requested fitzgen for a review on PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:36):

alexcrichton requested wasmtime-core-reviewers for a review on PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:36):

alexcrichton requested wasmtime-default-reviewers for a review on PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:37):

alexcrichton updated PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:39):

alexcrichton updated PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:58):

cfallin requested cfallin for a review on PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 20:59):

cfallin commented on PR #11495:

Happy to take this one for review (and thanks for doing this!). Working out a fix to #11489 which I'll want to make sure doesn't conflict (probably won't?) then will review this.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:16):

alexcrichton updated PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:24):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:24):

fitzgen created PR review comment:

:eyes:

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:33):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:33):

cfallin created PR review comment:

I believe this is an ET jmp, for reference

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:48):

alexcrichton updated PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:48):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:48):

alexcrichton created PR review comment:

One day we'll have support for that in Wasmtime...

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:57):

cfallin submitted PR review:

Looks great, thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:57):

cfallin created PR review comment:

s/as/has/

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:59):

alexcrichton updated PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 21:59):

alexcrichton has enabled auto merge for PR #11495.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 21 2025 at 22:42):

alexcrichton merged PR #11495.


Last updated: Dec 06 2025 at 07:03 UTC