abrown submitted PR Review.
abrown created PR Review Comment:
Figured this out: the
uload*
instructions need to have encodings not just for thefld
recipe but alsofldDisp8
andfldDisp32
.
abrown updated PR #1208 from load-extend-rebased
to master
:
This change adds Wasm SIMD [load extend] functionality to the x86 backend by adding new Cranelift instructions for each Wasm SIMD instruction. This closes #1176.
[load extend]: https://github.com/WebAssembly/simd/blob/master/proposals/simd/SIMD.md#load-and-extend
abrown requested julian-seward1 for a review on PR #1208.
abrown updated PR #1208 from load-extend-rebased
to master
:
This change adds Wasm SIMD [load extend] functionality to the x86 backend by adding new Cranelift instructions for each Wasm SIMD instruction. This closes #1176.
[load extend]: https://github.com/WebAssembly/simd/blob/master/proposals/simd/SIMD.md#load-and-extend
julian-seward1 submitted PR Review.
julian-seward1 submitted PR Review.
abrown submitted PR Review.
abrown created PR Review Comment:
I think you are right, it would be more explicit but that would be inconsistent with
sload8
,uload16
, etc.
abrown submitted PR Review.
abrown created PR Review Comment:
I actually removed this commit and @bnjbvr is reviewing it separately in https://github.com/bytecodealliance/wasmtime/pull/1191.
abrown updated PR #1208 from load-extend-rebased
to master
:
This change adds Wasm SIMD [load extend] functionality to the x86 backend by adding new Cranelift instructions for each Wasm SIMD instruction. This closes #1176.
[load extend]: https://github.com/WebAssembly/simd/blob/master/proposals/simd/SIMD.md#load-and-extend
abrown merged PR #1208.
Last updated: Nov 22 2024 at 17:03 UTC