Skip to content

reference types: Implement the table.size and table.grow instructions#1894

Merged
alexcrichton merged 1 commit into
bytecodealliance:masterfrom
fitzgen:table-size-and-table-grow
Jun 18, 2020
Merged

reference types: Implement the table.size and table.grow instructions#1894
alexcrichton merged 1 commit into
bytecodealliance:masterfrom
fitzgen:table-size-and-table-grow

Conversation

@fitzgen
Copy link
Copy Markdown
Member

@fitzgen fitzgen commented Jun 17, 2020

(Still doesn't handle table.grow for funcrefs yet, as there are still some open questions about how to represent on-stack, in-Wasm funcrefs.)

Also de-duplicates / cleans up declaring builtin function indices a little bit, so that they aren't so tedious and repetitive.

Part of #929

@fitzgen fitzgen requested a review from alexcrichton June 17, 2020 21:06
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Jun 17, 2020
@github-actions
Copy link
Copy Markdown

Subscribe to Label Action

cc @peterhuene

Details This issue or pull request has been labeled: "wasmtime:api"

Thus the following users have been cc'd because of the following labels:

  • peterhuene: wasmtime:api

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

Comment thread crates/environ/src/func_environ.rs Outdated
Comment thread crates/runtime/src/libcalls.rs
@fitzgen fitzgen force-pushed the table-size-and-table-grow branch from a2740f0 to 6bc16b0 Compare June 17, 2020 22:39
@fitzgen
Copy link
Copy Markdown
Member Author

fitzgen commented Jun 17, 2020

@alexcrichton ok, feedback addressed 👍

@fitzgen fitzgen force-pushed the table-size-and-table-grow branch from 6bc16b0 to f8419f0 Compare June 17, 2020 23:03
@alexcrichton alexcrichton merged commit bbd99c5 into bytecodealliance:master Jun 18, 2020
@fitzgen fitzgen deleted the table-size-and-table-grow branch June 18, 2020 15:44
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