Skip to content

Conversation

StephanTLavavej
Copy link
Member

@StephanTLavavej StephanTLavavej commented Mar 18, 2022

This updates the Azure VMSS pool to StlBuild-2022-03-17-T1445, containing March 2022's Patch Tuesday, VS 2022 17.2 Preview 2, and Python 3.10.3. (Built with PowerShell 7.2.2.)

The CMake/Ninja versions are unchanged. The README still mentions Python 3.9 as the minimum, as that's what the VS Installer still offers. This is not increasing the minimum _MSC_VER required as the internal build still uses an older compiler version.

Since switching to Server 2022 in #2496, we've had remarkably well-behaved checks, with virtually no stalled checks. I'd like to try removing the hack of using N - 2 for test parallelism (i.e. 30 for our 32-core VMs).

This removes a number of compiler bug workarounds:

  • DevCom-1162647 (constexpr variant stores wrong pointer) was fixed in 17.1.
  • VSO-1466711 "/scanDependencies generates incorrect "logical-name" in JSON output" was fixed in 17.2 Preview 2.
  • VSO-1471374 "Standard Library Header Units: Deduplication emits fatal error C1116: unrecoverable error importing module, with <concepts>" was fixed in 17.2 Preview 2.
  • VSO-1471382 "Standard Library Header Units: Deduplication emits error C2672: 'count_if': no matching overloaded function found" was fixed in 17.2 Preview 2.

Additionally, 17.2 Preview 2 added _rotl8, _rotl16, _rotr8, _rotr16 to <intrin0.h>, so @AlexGuteniev's #2369 can be completed. Note that (unlike the 32/64-bit intrinsics) their second parameters are declared as unsigned char so we need to static_cast. (Fortunately, the intrinsics have modulo behavior.)

Finally, this adds two unrelated small cleanups found while working on Standard Library Modules:

  • Drop duplicate using _CSTD FILE; in <cstdio>.
  • Centralize using-declarations in <csignal>.

StephanTLavavej and others added 11 commits March 17, 2022 14:43
This was "`/scanDependencies` generates incorrect `"logical-name"` in JSON output".
This was "Standard Library Header Units: Deduplication emits `fatal error C1116: unrecoverable error importing module`, with `<concepts>`".
This was "Standard Library Header Units: Deduplication emits `error C2672: 'count_if': no matching overloaded function found`".
@StephanTLavavej StephanTLavavej added the infrastructure Related to repository automation label Mar 18, 2022
@StephanTLavavej StephanTLavavej marked this pull request as ready for review March 18, 2022 04:24
@StephanTLavavej StephanTLavavej requested a review from a team as a code owner March 18, 2022 04:24
@StephanTLavavej StephanTLavavej self-assigned this Mar 18, 2022
@StephanTLavavej
Copy link
Member Author

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

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.

2 participants