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.

Sign up for GitHub

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

Jump to bottom

Toolset update: VS 2022 17.13 Preview 2 #5185

Closed
wants to merge 7 commits into from
Closed

Toolset update: VS 2022 17.13 Preview 2 #5185

wants to merge 7 commits into from

Conversation

StephanTLavavej
Copy link
Member

@StephanTLavavej StephanTLavavej commented Dec 12, 2024 •
edited
Loading

📜 Changelog

  • Code cleanups:
    • Removed compiler bug workarounds.
  • Infrastructure improvements:
    • Updated dependencies.
      • Updated build compiler to VS 2022 17.13 Preview 2.
      • Updated Python to 3.13.1.

⚙️ Commits

  • Remove workarounds for:
    • VSO-2188364 "EDG assertion failed in conversion_for_direct_reference_binding_possible".
    • VSO-2254804 "EDG ICE in cpfe.dll!make_coroutine_result_expression with C++23 <generator> test".
    • VSO-2046190 "[CI-NIGHTLY][Libs-ASan-amd64] src\vctools\crt\github\tests\std\tests\Dev09_056375_locale_cleanup failed due to ERROR: AddressSanitizer: access-violation on unknown address".
  • Add /shallowScan to work around VSO-2293247 "/Zc:preprocessor does not terminate macro definitions properly".
    • This has been fixed internally, so we don't need to mirror this to the Perl script.
  • Python 3.13.1.
  • Standard_D32ads_v5 pool.
  • VS 2022 17.13 Preview 2 (not yet required).

STL-ASan-CI passed.

@StephanTLavavej StephanTLavavej added infrastructure Related to repository automation uncharted Excluded from the Status Chart labels Dec 12, 2024
@StephanTLavavej StephanTLavavej requested a review from a team as a code owner December 12, 2024 04:18
@StephanTLavavej StephanTLavavej removed the uncharted Excluded from the Status Chart label Dec 12, 2024
@StephanTLavavej

This comment was marked as resolved.

Sign in to view
@azure-pipelines Azure Pipelines

This comment was marked as resolved.

Sign in to view
@StephanTLavavej StephanTLavavej added the uncharted Excluded from the Status Chart label Dec 12, 2024
@StephanTLavavej
Copy link
Member Author

Superseded by #5186.

@StephanTLavavej StephanTLavavej deleted the vs17.13p2 branch December 12, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CaseyCarter CaseyCarter CaseyCarter approved these changes

Assignees
No one assigned
Labels
infrastructure Related to repository automation uncharted Excluded from the Status Chart
Projects
STL Code Reviews
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@StephanTLavavej @CaseyCarter