This repository was archived by the owner on Dec 22, 2021. It is now read-only.
BinarySIMD.md doesn't document representation of LaneIdxN #242
Comments
|
@ngzhian Thanks for the PR! FWIW your PR auto-closed this issue, but the last part from the description:
still remains. |
ngzhian
added a commit
to ngzhian/simd
that referenced
this issue
Jun 10, 2020
Fixed WebAssembly#242. (again)
|
Ah! #253 should fix this, thanks! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The SIMD instruction encodings table in BinarySIMD.md references LaneIdx32, LaneIdx16, etc., but doesn't define their representation anywhere.
I've been pointed to https://github.com/WebAssembly/simd/blob/master/proposals/simd/SIMD.md#immediate-operands which does mention all of these are just single bytes, but it would be useful to encode this in the binary representation document as well.
Additionally, the name in the explainer (
ImmLaneIdxN) mismatches the one used in BinarySIMD (LaneIdxN), leading to further potential confusion.The text was updated successfully, but these errors were encountered: