-
Notifications
You must be signed in to change notification settings - Fork 186
Add memory64 row to feature table #248
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
Conversation
Hmm I'm not sure if it's very helpful if the entire row is "unsupported" anyway. I'd rather include it once we have some engine or browser info to accompany it. |
Yeah, makes sense. I guess I was interested to see what other engines were doing since Wasmtime may get support soon: bytecodealliance/wasmtime#3153. Maybe it makes sense to add this row after that PR is merged? |
We note feature support only for actual (stable) releases of engines, but yeah, once it's released in Wasmtime, we can add it. |
Do you know what the status is for this in other engines, by chance? |
No idea, sorry. But I don't think it's implemented anywhere yet. |
@abrown @RReverser |
Do we have a flag to enable them yet that we can document here? Or is it still early stages? |
Early stages, I think. |
In that case let's keep this open for now till we have any flags we can provide for users to experiment with. |
In Wasmtime, the flag will be --enable-memory64. |
@abrown Is it supported in Wasmtime stable now? It appears the linked merged commit bytecodealliance/wasmtime@e0c8961 was released in 0.31.0. |
cc @fitzgen too for the question above |
Yes, it is in 0.31.0 |
Thanks. Anything else that landed since last update? Do you perhaps want to make a PR or should I? (happy to add this myself if this is the only change) |
I think that's it! So if you could add it, that would be very appreciated :) |
Added to this PR. |
I had not been in the loop about the status of the memory64 proposal; I was pointed to https://webassembly.org/roadmap and, not seeing it there, thought it would be helpful to add a row for it.