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

Intel intrinsics and SIMDe #1160

Open
smoe opened this issue Feb 27, 2021 · 3 comments
Open

Intel intrinsics and SIMDe #1160

smoe opened this issue Feb 27, 2021 · 3 comments

Comments

@smoe
Copy link
Contributor

smoe commented Feb 27, 2021

Dear Alex,

I was a bit too quick when I uploaded 2.7.8a to Debian - the Intel-specific instructions prohibited a smooth build on other platforms. Now, a contribution from Debian Med had prepared a patch to use https://github.com/simd-everywhere/simde which is https://salsa.debian.org/med-team/rna-star/-/blob/master/debian/patches/simde.patch .

Since Debian is about to freeze, and conda and guix already ship 2.7.8a, is why I was so quick and this is also why we are now just a bit nervous that we are doing the right thing. With that patch, the package locally builds (and autotests) on all platforms, so we would be good to go, this is unless you are objecting to the idea to use simde. In the contrary, if you happen to like this, you may then want to adopt that patch to keep ARM in particular in the loop.

Many thanks and regards,
Steffen

@alexdobin
Copy link
Owner

Hi Steffen,

thanks - It's a great idea to start taking care of ARMs.
I will look into simde more closely. It seems that SIMDE_ENABLE_NATIVE_ALIASES is not a generally recommended solution, but since it works we can go with it for now.
The simd instructions are used in the OPAL package, so changing the APIs to simde will take some work.

Cheers
Alex

@mr-c
Copy link
Contributor

mr-c commented Mar 1, 2021

It seems that SIMDE_ENABLE_NATIVE_ALIASES is not a generally recommended solution

Hello @alexdobin ; I'm a co-maintainer for SIMDe. We say that out of an abundance of caution; for this codebase it should be fine. Would you like a PR to add SIMDe?

@smoe
Copy link
Contributor Author

smoe commented Mar 1, 2021

@mr-c is another developer for Debian ... did not know about your upstream work on SIMDe. Great! :o)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants