Skip to content

Fill out MAY_REQUIRE_REALLOC for more types#13158

Merged
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:may-require-alloc
Apr 21, 2026
Merged

Fill out MAY_REQUIRE_REALLOC for more types#13158
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:may-require-alloc

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This is an relatively impactful optimization to the performance of futures/streams when transferring data from the host to the guest, so fill this out for more types get the optimization to fire more. This was found during some improvement of wasi-libc where some code paths weren't getting hit, but they should,, and this is required to have that work out.

This is an relatively impactful optimization to the performance of
futures/streams when transferring data from the host to the guest, so
fill this out for more types get the optimization to fire more. This was
found during some improvement of wasi-libc where some code paths weren't
getting hit, but they should,, and this is required to have that work out.
@alexcrichton alexcrichton requested a review from dicej April 21, 2026 01:36
@alexcrichton alexcrichton requested a review from a team as a code owner April 21, 2026 01:36
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Apr 21, 2026
@alexcrichton alexcrichton added this pull request to the merge queue Apr 21, 2026
Merged via the queue into bytecodealliance:main with commit eb8e58d Apr 21, 2026
48 checks passed
@alexcrichton alexcrichton deleted the may-require-alloc branch April 21, 2026 14:50
alexcrichton added a commit to alexcrichton/wasi-libc that referenced this pull request Apr 21, 2026
Set the flags that an accept is in-progress regardless of whether it
completes or not, resolving an assertion failure. This is not currently
possible to test with Wasmtime, but with bytecodealliance/wasmtime#13158
and the test included here the assertion is hit reliably.
alexcrichton added a commit to WebAssembly/wasi-libc that referenced this pull request Apr 21, 2026
Set the flags that an accept is in-progress regardless of whether it
completes or not, resolving an assertion failure. This is not currently
possible to test with Wasmtime, but with bytecodealliance/wasmtime#13158
and the test included here the assertion is hit reliably.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants