-
Notifications
You must be signed in to change notification settings - Fork 107
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
Update and evaluate wasmi
#3449
Comments
More precisely, it requires no memory that is both writable and executable. It is fine (in fact, necessary) to have executable memory. Also to add to the list of requirements is that it must work on a single threaded environment and without async. |
For the record, I believe you were suggesting looking at https://github.com/bytecodealliance/wasmtime , right? |
As wasmi does not support calling Wasm functions from host functions after the update (cf. github.com/paritytech/wasmi/issues/572), our current implementation of |
Options to mitigate:
|
With the wasmi update we should revisit whether our original assumptions and reasons for choosing wasmi still hold.
Our reasons for wasmi:
no_std
The text was updated successfully, but these errors were encountered: