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: Component Model Support #8087

Closed
saulecabrera opened this issue Mar 12, 2024 · 1 comment
Closed

winch: Component Model Support #8087

saulecabrera opened this issue Mar 12, 2024 · 1 comment
Assignees
Labels
winch Winch issues or pull requests

Comments

@saulecabrera
Copy link
Member

In order for Winch to reach feature parity with Cranelift, we want to ensure that it can compile WebAssembly Components.

One approach is to implement the component trampolines from scratch, however, this will likely be a considerable amount of work and a lot of very similar work will be duplicated across both compilers. An alternative is to using Cranelift to compile all the trampolines, including the component ones. This approach will require making Winch match Cranelift's ABI. At that time of writing, this mostly involves adding callee-saved registers to Winch's ABI.

Switching to using Cranelift's trampolines has the added benefit that Winch will receive support for all trampolines types for "free".

@saulecabrera saulecabrera added the winch Winch issues or pull requests label Mar 12, 2024
@elliottt
Copy link
Member

Fixed in #8082

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

No branches or pull requests

2 participants