github-actions[bot] commented on issue #3123:
Subscribe to Label Action
cc @fitzgen
<details>
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:
- fitzgen: cranelift:area:peepmatic
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
alexcrichton commented on issue #3123:
Also, while here, I went ahead and updated the release notes -- https://github.com/bytecodealliance/wasmtime/commit/e9fe64f0c7eb14975f1788a57b097258ad8850d0 -- mind including that here?
cfallin commented on issue #3123:
Ah, I hadn't done that yet, thanks!
jlb6740 commented on issue #3123:
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:
match (testsuite, testname) { ("simd", "simd_i16x8_extadd_pairwise_i8x16") => return true, ("simd", "simd_i32x4_extadd_pairwise_i16x8") => return true, ("simd", "simd_int_to_int_extend") => return true, ("simd", _) => return false, _ => {} }
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.
cfallin commented on issue #3123:
I just updated this PR to rebase on top of latest
main
with the last x64 SIMD bits (and Alex's just-merged 64-bit memory limit PR). Also updated release notes accordingly. PTAL @alexcrichton for updated note on the latter especially -- thanks!
Last updated: Nov 22 2024 at 17:03 UTC