Skip to content

Make most tests pass with /Zc:alignedNew- - #6087

Merged
Stephan T. Lavavej (StephanTLavavej) merged 14 commits into
microsoft:mainfrom
AlexGuteniev:misalign
Mar 4, 2026
Merged

Make most tests pass with /Zc:alignedNew-#6087
Stephan T. Lavavej (StephanTLavavej) merged 14 commits into
microsoft:mainfrom
AlexGuteniev:misalign

Conversation

@AlexGuteniev

@AlexGuteniev Alex Guteniev (AlexGuteniev) commented Feb 15, 2026

Copy link
Copy Markdown
Contributor

Towards #2713 whatever it now means.

Make sure tests pass with /Zc:alignedNew-, and then revert matrices back.

The following C++17 features do not pass:

  • P0035R4_over_aligned_allocation. Well, that's it.
  • P0220R1_any. I don't know if the feature itself works. Test relies on libc++ which does not assume any escape hatches.
  • P0220R1_polymorphic_memory_resources. It appears to be partially broken both as a feature and the tests.
    • Additionally, a part of <generator> test that uses pmr had to be disabled.

Comment thread tests/std/tests/GH_005504_avoid_function_call_wrapping/test.cpp Outdated
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the enhancement Something can be improved label Feb 15, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added test Related to test code and removed enhancement Something can be improved labels Feb 15, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Work In Progress in STL Code Reviews Feb 15, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Work In Progress to Initial Review in STL Code Reviews Feb 15, 2026
@AlexGuteniev Alex Guteniev (AlexGuteniev) changed the title Use /Zc:alignedNew- for the latest features Make most tests pass with /Zc:alignedNew- Feb 15, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Mar 3, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Mar 3, 2026
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo. Please notify me if any further changes are pushed, otherwise no action is required.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 3a8becf into microsoft:main Mar 4, 2026
45 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Mar 4, 2026
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for investigating this! 🐈 🐈‍⬛ 🐈

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.

2 participants