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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<string>: wstring::find_first_of crashes on some inputs

3 participants