-
Notifications
You must be signed in to change notification settings - Fork 2
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
Enable sse2/avx2 if available #7
Conversation
@LDeakin Does this PR work for you in getting the intrinsics version of shuffles? This requires |
f1fce0a
to
07cf938
Compare
07cf938
to
a15c3b2
Compare
Yes, that does the trick! Although, I don't even need to set |
Update: with |
Yeah, I see the crash when using |
You probably need these compile flags https://github.com/Blosc/c-blosc/blob/bdbb700787a2485a756eea63ced268f9a00f6370/blosc/CMakeLists.txt#L157-L192 |
Need to pass these flags to |
3290ebd works and I've tested it with a suite of external benchmarks, LGTM. |
Thanks for picking up that this crate did not use SSE2/AVX @LDeakin. I will release a new version later today |
No worries, thanks for addressing it so quickly! |
mulimoen/rust-blosc-src#7 Bump blosc-sys to 0.3.0
No description provided.