-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Bump to Wasmtime v0.29.0 and Cranelift 0.76.0. #3123
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me, but as you mentioned on Zulip let's wait to hear from @jlb6740. If simd might be done by the end of the week I think it might make sense to squeeze that in before the release
Subscribe to Label Actioncc @fitzgen
This issue or pull request has been labeled: "cranelift", "cranelift:area:peepmatic", "cranelift:meta", "cranelift:module", "cranelift:wasm"
Thus the following users have been cc'd because of the following labels:
To subscribe or unsubscribe from this label, edit the |
Also, while here, I went ahead and updated the release notes -- e9fe64f -- mind including that here? |
Ah, I hadn't done that yet, thanks! |
Hi @cfallin @alexcrichton .. Yes there are two more pull requests that are needed to complete SIMD. Specifically what is left is what covers these tests:
The first two are taken care of by pr #3031. It actually works fine but I think we want to do a little refactoring. The lowerings that cover int_to_int_extend I haven't pushed a pr for but have started. Let me see how far I get and give an updated estimate later this week if no urgency to finish by this week. |
I just updated this PR to rebase on top of latest |
Release Wasmtime v0.29.0 and Cranelift v0.76.0.