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

[C++] Provide non-SSE4 versions of functions that use CPU intrinsics for older processors #16174

Closed
asfimport opened this issue Feb 6, 2017 · 3 comments

Comments

@asfimport
Copy link
Collaborator

follow up to discussion in patch for ARROW-33

Reporter: Wes McKinney / @wesm
Assignee: Antoine Pitrou / @pitrou

Note: This issue was originally created as ARROW-536. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Uwe Korn / @xhochy:
The current code (as in ARROW-33) will compile and work on non-SSE4 system but the popcnt instruction will only be used by the compiler if the flag -msse4.2 is provided. We would need rather two variants of specific functions compiled, one with SSE4.2 instructions and one without. Loading the correct one must be resolved at runtime.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
As part of ARROW-2653 I am making SSE4.2 mandatory on x86-64 systems. So this issue can probably be closed once the PR gets merged.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
+1

@asfimport asfimport added this to the 0.12.0 milestone Jan 11, 2023
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

2 participants