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(x64): SIMD #8093

Open
saulecabrera opened this issue Mar 12, 2024 · 7 comments
Open

winch(x64): SIMD #8093

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

Comments

@saulecabrera
Copy link
Member

saulecabrera commented Mar 12, 2024

Winch is missing support for most of finished WebAssembly proposals. The goal of this issue is to track support for the SIMD proposal

@saulecabrera saulecabrera converted this from a draft issue Mar 12, 2024
@saulecabrera saulecabrera added the winch Winch issues or pull requests label Mar 12, 2024
@elliottt
Copy link
Member

Once we're working SIMD support, the cranelift Winch calling convention will need to be updated with whatever mechanism we choose. There's a TODO in the x64 abi introduced in #8210 that points out where we'll need to handle this for return values.

@MarinPostma
Copy link
Contributor

i'll take the v128 bunch.

@MarinPostma
Copy link
Contributor

I'll just take the rest

@saulecabrera
Copy link
Member Author

@MarinPostma I know @jeffcharles has plans to work on some of the unassigned instructions as well, so to keep the list tight and avoid duplicated effort, maybe we keep assigning them in batches?

@MarinPostma
Copy link
Contributor

no problem, then I'll take add, sub and mul for packed integer types, to begin with :)

@MarinPostma
Copy link
Contributor

I'll do shifts, neg, min and max next

@MarinPostma
Copy link
Contributor

I'll take the extmul/add next.

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: In Progress
Development

No branches or pull requests

3 participants