Skip to content

Adjust some behavior/docs around table/memory growth - #14589

Merged
alexcrichton merged 2 commits into
bytecodealliance:mainfrom
alexcrichton:tweak-growth
Oct 7, 2026
Merged

alexcrichton merged 2 commits into
bytecodealliance:mainfrom
alexcrichton:tweak-growth

Conversation

@alexcrichton

Copy link
Copy Markdown
Member
  • Document that *_grow_failed methods might be invoked without ever calling *_growing due to size constraints.
  • Adjust table memory allocation failures to go into wasm as a -1 return value of table.grow rather than a trap.

* Document that `*_grow_failed` methods might be invoked without ever
  calling `*_growing` due to size constraints.
* Adjust table memory allocation failures to go into wasm as a -1 return
  value of `table.grow` rather than a trap.
@alexcrichton
alexcrichton requested a review from a team as a code owner October 6, 2026 20:46
@alexcrichton
alexcrichton requested review from pchickey and removed request for a team October 6, 2026 20:46
@alexcrichton
alexcrichton added this pull request to the merge queue Oct 6, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 6, 2026
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Oct 6, 2026
OOMs are just hard aborts...
@alexcrichton
alexcrichton enabled auto-merge October 6, 2026 23:51
@alexcrichton
alexcrichton added this pull request to the merge queue Oct 6, 2026
Merged via the queue into bytecodealliance:main with commit 1640031 Oct 7, 2026
53 checks passed
@alexcrichton
alexcrichton deleted the tweak-growth branch October 7, 2026 00:18
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