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

SSSE3 checks #12

Closed
hellrot opened this issue May 24, 2019 · 2 comments
Closed

SSSE3 checks #12

hellrot opened this issue May 24, 2019 · 2 comments

Comments

@hellrot
Copy link

hellrot commented May 24, 2019

There are two places protected with a check of PHMAP_HAVE_SSSE3, both in phmap.h. The protected intrinsics, _mm_movemask_epi8 and _mm_or_si128, are both SSE2 intrinsics.

It seems you could just remove those checks, the corresponding the #else case code, and simplify phmap_config.h to only test for (and require) SSE2.

@greg7mdp
Copy link
Owner

Thanks for the suggestion, but these two places also require _mm_sign_epi8 and _mm_shuffle_epi8 which are SSE3 if I'm not mistaken.

@hellrot
Copy link
Author

hellrot commented May 24, 2019

Oops. You are correct, I overlooked those.

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

2 participants