-
Notifications
You must be signed in to change notification settings - Fork 42
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
Wasm64 support #74
Comments
AFAIK WASI doesn't have a wasm64 version. How did you compile this wasm module? |
Emscripten. I didn't realize WASI didn't support wasm64. Are there any alternative standard interfaces? |
Emscripten originally used a fully custom ABI, but moved to a subset of WASI + extra methods for functionality not part of WASI. Wasm64 support for WASI is probably an Emscripten specific extensions. I'm curious why you are trying to use browser_wasi_shim with an Emscripten compiled wasm module though. Emscripten already emits javascript for handling their entire ABI including all those extra methods and extensions. |
I've been using the WASI api (using Emscripten's glue code generation creates a lot of stuff I don't necessarily want. I'm using Nim->C->Wasm. So the resulting Emscripten glue code for passing data to and from JS ends up being pretty mismatched with Nim semantics. At the point that I need to pass data to and from JS, I'm just going to write the JS code myself to match the semantics I want in a perf-conscious way. I've experimented with this a bit and I much prefer it to Emscripten's generated stuff. So I may be heading in an unusual direction and you don't want to support 64-bit memory because it would entail a bunch of extra code for something that may or may not see much use in the near term. That makes sense. I just wanted to flag this as a limitation. |
I see. I've asked on the wasm discord if there is any documentation about the wasm64 extension to WASI of Emscripten. I'm not sure yet if I want to support wasm64. I think I will have to try implementing it once I know what to actually implement to see how much extra code it needs. |
Got a reply. Seems like support for the combination of PURE_WASI and MEMORY64 is purely accidental and the resulting ABI is not documented. |
Okay. Good to know. Thanks for your time with all this! |
Hi! Thanks for this great library!
Just a heads up that I've run into an issue around BigInts when using 64-bit wasm. I know this is to be expected but figured I'd flag it all the same just to get it on your radar.
The text was updated successfully, but these errors were encountered: