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

transpile: Add SIMD intrinsic __builtin_ia32_vperm2f128_pd256/_mm256_permute2f128_pd #1129

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Kriskras99
Copy link
Contributor

No description provided.

@fw-immunant
Copy link
Contributor

Thanks for the contribution!

This looks sane, but it would be nice to have a bit of background: is there an example .c file that can be used to validate this, and should we add support for other AVX intrinsics at the same time?

@kkysen kkysen changed the title Add SIMD intrinsic __builtin_ia32_vperm2f128_pd256/_mm256_permute2f128_pd transpile: Add SIMD intrinsic __builtin_ia32_vperm2f128_pd256/_mm256_permute2f128_pd Sep 18, 2024
@Kriskras99
Copy link
Contributor Author

I ran into this when transpiling soxr. This was the only intrinsic missing for that code base.

I can extract an example usage from that code base to use as a test

@Kriskras99
Copy link
Contributor Author

Also seems the Github Action needs to be updated

@kkysen
Copy link
Contributor

kkysen commented Sep 22, 2024

I can extract an example usage from that code base to use as a test

Yeah, a simple test for this would be nice. Thanks!

@Kriskras99 Kriskras99 force-pushed the add_intrinsic_vperm2f128_pd256 branch from 7e2bc29 to 2bd33e1 Compare October 23, 2024 22:23
@Kriskras99
Copy link
Contributor Author

I've added a test for the intrinsic, and it passes through the CI.

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.

3 participants