Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wit-bindgen support for value imports and value exports #263

Closed
kulakowski-wasm opened this issue Jun 24, 2022 · 3 comments
Closed

Wit-bindgen support for value imports and value exports #263

kulakowski-wasm opened this issue Jun 24, 2022 · 3 comments

Comments

@kulakowski-wasm
Copy link
Contributor

No description provided.

@esoterra
Copy link
Contributor

I think the mechanism for complex (e.g. allocation-requiring) value imports will need to be discussed and rendered in the component model Canonical ABI before it can be implemented here. Particularly, this should address the initialization ordering of value imports, and what the realloc function called during lowering may assume has already been initialized.

@sunfishcode
Copy link
Member

WebAssembly/component-model#71 describes a specific ABI for value imports and exports.

@alexcrichton
Copy link
Member

I'm going to close this for now since given the discussion on WebAssembly/component-model#100 the conclusion seems to be that values/start/etc should be postponed to a later date when other aspects are more fleshed out in the component model. To that end I think that short-to-mid-term designs should assume that value imports/exports and the start function are not going to be implemented and not supported.

In the near-to-mid-term my hope is that wit-bindgen can be aligned with the component model as specified so I don't believe that this is going to make the cut since it can't be implemented in Wasmtime at the host level yet due to spec/architectural issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

No branches or pull requests

4 participants