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.

Sign up for GitHub

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

Jump to bottom

<algorithm>: vectorize min_element / max_element using SSE4.1/AVX for floats #2439

Closed
AlexGuteniev opened this issue Dec 20, 2021 · 1 comment · Fixed by #3928
Closed

<algorithm>: vectorize min_element / max_element using SSE4.1/AVX for floats #2439

AlexGuteniev opened this issue Dec 20, 2021 · 1 comment · Fixed by #3928
Labels
fixed Something works now, yay! performance Must go faster

Comments

@AlexGuteniev
Copy link
Contributor

Similar to #2438 .

I assume it is safe for /fp:fast. Not sure about other FP modes and how to handle FP corner cases such as denormals or singalings.
@StephanTLavavej StephanTLavavej added the performance Must go faster label Jan 12, 2022
@StephanTLavavej StephanTLavavej mentioned this issue Jun 18, 2022
Closed
3 tasks
@AlexGuteniev AlexGuteniev mentioned this issue Jun 24, 2022
Merged
@AlexGuteniev
Copy link
Contributor Author

Will need to be careful about randomized test with a distribution that indeed has coverage

AlexGuteniev added a commit to AlexGuteniev/STL that referenced this issue Aug 6, 2023
@AlexGuteniev
vectorize min/max_element using SSE4.1/AVX for floats
cd7420a 
Resolves microsoft#2439
@AlexGuteniev AlexGuteniev mentioned this issue Aug 6, 2023
Merged
@StephanTLavavej StephanTLavavej added the fixed Something works now, yay! label Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fixed Something works now, yay! performance Must go faster
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

vectorize min/max_element using SSE4.1 for floats AlexGuteniev/STL
2 participants
@AlexGuteniev @StephanTLavavej