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 Core Wasm Opcode Support #6528

Closed
saulecabrera opened this issue Jun 6, 2023 · 3 comments · Fixed by #7894
Closed

Winch Core Wasm Opcode Support #6528

saulecabrera opened this issue Jun 6, 2023 · 3 comments · Fixed by #7894
Labels
winch Winch issues or pull requests

Comments

@saulecabrera
Copy link
Member

saulecabrera commented Jun 6, 2023

During the debugging meeting on June 1, 2023, we discussed opening an issue to track the missing Core Wasm opcodes in Winch. The objective of this issue is to make it easier for potential contributors to know where to start.

This list is intended to be a comprehensive compilation of the missing opcodes required to achieve parity with Core Wasm(*). Each entry also includes an owner, indicating that work on a particular opcode is currently in progress.

(*) Excluding SIMD and Reference Types to start, as drafted in the initial RFC. We can open dedicated issues as we get to those and other proposals.

cc/ @fitzgen @itsrainy

Proposals

@saulecabrera saulecabrera added the winch Winch issues or pull requests label Jun 6, 2023
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Subscribe to Label Action

cc @saulecabrera

This issue or pull request has been labeled: "winch"

Thus the following users have been cc'd because of the following labels:

  • saulecabrera: winch

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@itsrainy
Copy link
Contributor

itsrainy commented Jun 6, 2023

This is great! Thanks for putting this together @saulecabrera!

Coincidentally @cfallin, @fitzgen and I started working on i32.popcnt and i64.popcnt just this morning, primarily as a way of giving me a mini-tour of winch. @jeffcharles if you've already done the work for those instructions, we can close the draft PR I just put up. It would also be helpful/interesting for me to compare implementations, if that's the case.

@jeffcharles
Copy link
Contributor

All good, I haven't started on either of those yet. I'm trying to debug an issue as part of implementing clz and ctz and I think your PR will be very helpful in resolving that issue.

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
None yet
Development

Successfully merging a pull request may close this issue.

3 participants