Skip to content

Toolset update: VS 2022 17.4 Preview 1#3017

Merged
StephanTLavavej merged 5 commits into
microsoft:mainfrom
StephanTLavavej:vs-17.4p1
Aug 12, 2022
Merged

Toolset update: VS 2022 17.4 Preview 1#3017
StephanTLavavej merged 5 commits into
microsoft:mainfrom
StephanTLavavej:vs-17.4p1

Conversation

@StephanTLavavej
Copy link
Copy Markdown
Member

  • Python 3.10.6.
  • New pool.
    • Contains Aug 2022's Patch Tuesday.
  • README.md: VS 2022 17.4 Preview 1, Ninja 1.11.0.
  • Require MSVC 19.34.
    • The MSVC-internal build is already using a checked-in 19.34 compiler.
  • Attempt to remove workaround for internal VSO-284079 "C1XX's C++14 constexpr emits bogus warnings C4146, C4308, C4307 for basic_string_view::iterator".
    • Although this bug hasn't been updated in 3 years, I observe that the warnings no longer repro. Let's try to activate this test coverage now.

@StephanTLavavej StephanTLavavej added the infrastructure Related to repository automation label Aug 10, 2022
@StephanTLavavej StephanTLavavej requested a review from a team as a code owner August 10, 2022 01:13
@strega-nil-ms strega-nil-ms self-assigned this Aug 10, 2022
Copy link
Copy Markdown
Contributor

@strega-nil-ms strega-nil-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Stephan!

@strega-nil-ms strega-nil-ms removed their assignment Aug 10, 2022
@StephanTLavavej StephanTLavavej self-assigned this Aug 11, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member Author

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

@StephanTLavavej StephanTLavavej merged commit 4c8d0ae into microsoft:main Aug 12, 2022
@StephanTLavavej StephanTLavavej deleted the vs-17.4p1 branch August 12, 2022 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Related to repository automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants