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

Remove need for SSE2 support #4

Open
satta opened this issue Aug 3, 2016 · 3 comments
Open

Remove need for SSE2 support #4

satta opened this issue Aug 3, 2016 · 3 comments

Comments

@satta
Copy link

satta commented Aug 3, 2016

Currently the code in ksw.c seems to need SSE2 to compile. It would be nice to have some kind of -- probably slower -- fallback implementation to improve support on architectures without these instructions.

@lh3
Copy link
Owner

lh3 commented Aug 4, 2016

I need to revamp ksw.c at some point. It is in fact buggy in rare conner cases. Will add non-SSE implementation when that day comes.

@satta
Copy link
Author

satta commented Aug 4, 2016

Thanks!

@clausecker
Copy link

clausecker commented Oct 15, 2024

As a stop-gap solution, you can use SSE2NEON to make it work on ARM by adding the header to the project and then applying this patch:

--- ksw.c.orig  2024-10-15 14:34:34 UTC
+++ ksw.c
@@ -25,7 +25,11 @@
 
 #include <stdlib.h>
 #include <stdint.h>
+#if defined(__arm__) || defined(__aarch64__)
+#include "sse2neon.h"
+#else
 #include <emmintrin.h>
+#endif
 #include "ksw.h"
 
 #ifdef __GNUC__

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

No branches or pull requests

3 participants