Stream: git-wasmtime

Topic: wasmtime / PR #10371 Fix an issue with DRC collector's ba...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 16:33):

fitzgen opened PR #10371 from fitzgen:externref-of-converted-i31-and-barriers to bytecodealliance:main:

Found during local fuzzing. This might be a fix for issue https://github.com/bytecodealliance/wasmtime/issues/10181 as well, but I want to get confirmation before marking it fixed, and IIUC there isn't a minimal test case in there at the moment.

<!--
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 (Mar 11 2025 at 16:33):

fitzgen requested dicej for a review on PR #10371.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 16:33):

fitzgen requested wasmtime-core-reviewers for a review on PR #10371.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 16:52):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 16:55):

fitzgen has enabled auto merge for PR #10371.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 17:00):

fitzgen updated PR #10371.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 17:10):

vouillon commented on PR #10371:

There was a reduced test case (https://github.com/bytecodealliance/wasmtime/issues/10181#issuecomment-2642608130), but it seems it is no longer crashing. The initial large test case is still crashing.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 17:13):

fitzgen commented on PR #10371:

@vouillon Thanks, looks like there are still some outstanding internal assertions in JIT code failing then. As always, a reduced test case would be very appreciated and would make debugging and fixing much easier!

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2025 at 17:36):

fitzgen merged PR #10371.


Last updated: Apr 18 2025 at 08:04 UTC