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

winch: memory64 #8089

Closed
saulecabrera opened this issue Mar 12, 2024 · 0 comments · Fixed by #8194
Closed

winch: memory64 #8089

saulecabrera opened this issue Mar 12, 2024 · 0 comments · Fixed by #8194
Labels
winch Winch issues or pull requests

Comments

@saulecabrera
Copy link
Member

Winch is missing support for most of the WebAssembly proposals supported by Cranelift (even the ones that are implemented and not enabled by default). The goal of this issue is to track support for the memory64 proposal. The bulk of the work here is to follow-up on the changes introduced in #7894

@saulecabrera saulecabrera added the winch Winch issues or pull requests label Mar 12, 2024
saulecabrera added a commit to saulecabrera/wasmtime that referenced this issue Mar 20, 2024
Closes: bytecodealliance#8089

This Winch unlocks support for the `memory64` proposal in Winch. After all the fixes to heap handling, all the spec and misc tests are passing, which is a good indication regarding the support for this proposal.

I'll like to merge this change after: bytecodealliance#8156.
saulecabrera added a commit to saulecabrera/wasmtime that referenced this issue Mar 20, 2024
Closes: bytecodealliance#8089

This commit  unlocks support for the `memory64` proposal in Winch. After all the fixes to heap handling, all the spec and misc tests are passing, which is a good indication regarding the support for this proposal.

I'll like to merge this change after: bytecodealliance#8156.
github-merge-queue bot pushed a commit that referenced this issue Mar 20, 2024
Closes: #8089

This commit  unlocks support for the `memory64` proposal in Winch. After all the fixes to heap handling, all the spec and misc tests are passing, which is a good indication regarding the support for this proposal.

I'll like to merge this change after: #8156.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
winch Winch issues or pull requests
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant