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.

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

Handle aligned vector<bool> in constexpr copy() properly #5347

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AlexGuteniev
Copy link
Contributor

Fixes #5345

@AlexGuteniev AlexGuteniev requested a review from a team as a code owner March 19, 2025 19:55
@StephanTLavavej StephanTLavavej added the bug Something isn't working label Mar 19, 2025
@StephanTLavavej StephanTLavavej changed the title Handle aligned vector<bool> in constexpr properly Handle aligned vector<bool> in constexpr copy() properly Mar 19, 2025
@StephanTLavavej StephanTLavavej self-assigned this Mar 19, 2025
@StephanTLavavej

This comment was marked as resolved.

@AlexGuteniev
Copy link
Contributor Author

I pushed a merge with main to pick up the libcxx update and unskip some tests that are now passing.

The reason of failure is now not correct for them.

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
Status: Initial Review
Development

Successfully merging this pull request may close these issues.

<vector>: _Copy_vbool() mishandles vector<bool>s of size 32 and 64, revealed by constexpr Clang
2 participants