Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@AlexGuteniev
Copy link
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.

StephanTLavavej reacted with heart emoji
@StephanTLavavejStephanTLavavej added the bugSomething isn't working labelOct 4, 2025
@StephanTLavavejStephanTLavavej self-assigned thisOct 4, 2025
@StephanTLavavejStephanTLavavej removed their assignmentOct 8, 2025
@StephanTLavavejStephanTLavavej moved this fromInitial Review toReady To Merge inSTL Code ReviewsOct 8, 2025
@StephanTLavavej
Copy link
Member

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

@StephanTLavavejStephanTLavavej moved this fromReady To Merge toMerging inSTL Code ReviewsOct 9, 2025
@StephanTLavavejStephanTLavavej merged commit5f63382 intomicrosoft:mainOct 10, 2025
39 checks passed
@github-project-automationgithub-project-automationbot moved this fromMerging toDone inSTL Code ReviewsOct 10, 2025
@StephanTLavavej
Copy link
Member

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

@AlexGutenievAlexGuteniev deleted the magic-it-happens branchOctober 10, 2025 08:50
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@StephanTLavavejStephanTLavavejStephanTLavavej approved these changes

+1 more reviewer

@AlcaroAlcaroAlcaro left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

bugSomething isn't working

Projects

Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

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

3 participants

@AlexGuteniev@StephanTLavavej@Alcaro
[8]ページ先頭
©2009-2025 Movatter.jp