Skip to content

The docs didn't state the v128.extract_lane's idx arg needs to compile time constant #2877

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.

Sign up for GitHub

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

Jump to bottom
Closed
hinum opened this issue Oct 21, 2024 · 2 comments · Fixed by #2890
Closed

The docs didn't state the v128.extract_lane's idx arg needs to compile time constant #2877

hinum opened this issue Oct 21, 2024 · 2 comments · Fixed by #2890
Labels
bug document good first issue

Comments

@hinum
Copy link

hinum commented Oct 21, 2024

Bug description

as the name suggested, the docs entry for v128.extract_lane didn't state that the idx argument need to be compile time constant

Steps to reproduce

  1. try to compile this
let a = 0
v128.extract_lane<u8>(v128.splat<u8>(0), a)
  1. compile error thrown

AssemblyScript version

not sure if relevant but 0.27.30
@hinum hinum added the bug label Oct 21, 2024
@HerrCai0907
Copy link
Member

related declarations are placed in std/assembly/index.d.ts.

KazChe added a commit to KazChe/assemblyscript that referenced this issue Dec 11, 2024
@KazChe
fix to enhance in-line comment for extract_lane functions to include …
509ffc4 
…idx argument needs to be compile time constant - issue AssemblyScript#2877
@KazChe
Copy link
Contributor

KazChe commented Dec 11, 2024

hover_test

@KazChe KazChe mentioned this issue Dec 11, 2024
Merged
KazChe added a commit to KazChe/assemblyscript that referenced this issue Dec 11, 2024
@KazChe
Revert "fix to enhance in-line comment for extract_lane functions to …
62425c3 
…include idx argument needs to be compile time constant - issue AssemblyScript#2877"

This reverts commit 509ffc4.
KazChe added a commit to KazChe/assemblyscript that referenced this issue Dec 11, 2024
@KazChe
Revert "Revert "fix to enhance in-line comment for extract_lane funct…
5604d81 
…ions to include idx argument needs to be compile time constant - issue AssemblyScript#2877""

This reverts commit 62425c3.
KazChe added a commit to KazChe/assemblyscript that referenced this issue Dec 11, 2024
@KazChe
applied inline comment for replace_lane functions: idx argument needs…
db38a87 
… to be compile time constant. - AssemblyScript#2877
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug document good first issue
Projects
None yet
Milestone
No milestone
3 participants
@KazChe @HerrCai0907 @hinum