Skip to content

vector_algorithms.cpp: fix incorrect activation of bitmap find-meow-of algorithms - #5758

Merged
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
AlexGuteniev:magic-it-happens
Oct 10, 2025
Merged

vector_algorithms.cpp: fix incorrect activation of bitmap find-meow-of algorithms#5758
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
AlexGuteniev:magic-it-happens

Conversation

@AlexGuteniev

Copy link
Copy Markdown
Contributor

Fix #5757

The activation function _Can_fit_256_bits_sse skipped past some elements, which caused incorrect activation of bitmap algorithm, which then addressed array out-of-bounds.

Comment thread tests/std/tests/VSO_0000000_vector_algorithms/test.cpp
Comment thread tests/std/tests/VSO_0000000_vector_algorithms/test.cpp
Comment thread tests/std/tests/VSO_0000000_vector_algorithms/test.cpp Outdated
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Oct 8, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Oct 9, 2025
@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 5f63382 into microsoft:main Oct 10, 2025
39 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Oct 10, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for investigating and fixing this regression so quickly! 🛠️ 🕵️ 🪄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

3 participants