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 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

2 participants