Skip to content
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

fix: Fix vpaddq for VLEN !=128 #398

Merged
merged 1 commit into from
Jun 4, 2024
Merged

fix: Fix vpaddq for VLEN !=128 #398

merged 1 commit into from
Jun 4, 2024

Conversation

camel-cdr
Copy link
Contributor

Turns out the vaddq changes in my last PR (#397) only work for VLEN=128, sorry.
I've reverted the changes, and added only the narrow instead of compress back in.
This now passes on qemu with all VLENs.

However the following functions still fail with VLEN=256:

Test vtbl1_s8                       failed
Test vtbl1_u8                       failed
Test vtbl2_s8                       failed
Test vtbl2_u8                       failed
Test vtbl3_s8                       failed
Test vtbl3_u8                       failed
Test vtbl4_s8                       failed
Test vtbl4_u8                       failed

Test vzipq_s8                       failed
Test vzipq_s16                      failed
Test vzipq_s32                      failed
Test vzipq_f32                      failed
Test vzipq_u8                       failed
Test vzipq_u16                      failed
Test vzipq_u32                      failed

Test vuzpq_s8                       failed
Test vuzpq_s16                      failed
Test vuzpq_s32                      failed
Test vuzpq_f32                      failed
Test vuzpq_u8                       failed
Test vuzpq_u16                      failed
Test vuzpq_u32                      failed

Test vst3_f32                       failed

@howjmay howjmay changed the title fix vpaddq for VLEN!=128 fix: Fix vpaddq for VLEN !=128 Jun 4, 2024
@howjmay
Copy link
Owner

howjmay commented Jun 4, 2024

for vlen != 128 I have opened an issue(#375), just I am busy for other things now.
Always more than happy for your help

@howjmay howjmay merged commit 2d38344 into howjmay:main Jun 4, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants