Skip to content

Add some coverage of constexpr vector<bool> - #5728

Merged
Stephan T. Lavavej (StephanTLavavej) merged 12 commits into
microsoft:mainfrom
AlexGuteniev:compile-time-boo
Oct 4, 2025
Merged

Add some coverage of constexpr vector<bool>#5728
Stephan T. Lavavej (StephanTLavavej) merged 12 commits into
microsoft:mainfrom
AlexGuteniev:compile-time-boo

Conversation

@AlexGuteniev

Copy link
Copy Markdown
Contributor

A. Jiang (@frederick-vs-ja) asked.

Things don't go smooth

@github-project-automation github-project-automation Bot moved this to Initial Review in STL Code Reviews Sep 17, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the test Related to test code label Sep 17, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Work In Progress in STL Code Reviews Sep 17, 2025
@AlexGuteniev

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Work In Progress to Initial Review in STL Code Reviews Sep 18, 2025
@AlexGuteniev

This comment was marked as resolved.

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

AraHaan commented Sep 29, 2025

Copy link
Copy Markdown

LGTM

Comment thread stl/inc/vector
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Oct 3, 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) merged commit 46ec4fc into microsoft:main Oct 4, 2025
39 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Oct 4, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

👻 👻 👻

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

Labels

test Related to test code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants