Skip to content

Conversation

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Aug 15, 2025

Re-lands #80122 but with release instead of debug for now because of the fatalError issue we're seeing. This still saves us 5-10 minutes on top of what we have today.

…Windows

We currently rebuild swift-syntax, swift-foundation-icu and swift-foundation twice: Once to test swift-foundation and once to test swift-corelibs-foundation. Using a unified build for both projects means that we only need to rebuild them once, saving ~5 minutes.

(cherry picked from commit 61f833c)
@bnbarham
Copy link
Contributor Author

@swift-ci please test Windows platform

@bnbarham bnbarham changed the title [DNM] Multiroot + release test [Windows] Use a multiroot data file to test (corelibs-)foundation on Windows Aug 16, 2025
@bnbarham bnbarham marked this pull request as ready for review August 16, 2025 00:33
@bnbarham bnbarham requested a review from jmschonfeld August 16, 2025 00:39
@bnbarham
Copy link
Contributor Author

Test-Foundation           Windows  x86_64  00:48:26.78  13.1% 

compared to the ~55 minutes we have at the moment

@bnbarham
Copy link
Contributor Author

Merging - this can only impact Windows and was already approved in #80122 (it just switched both to debug as well). I have #83772 up to try switching back to debug with a lower job count to see timings there.

@bnbarham bnbarham merged commit 375a6ed into swiftlang:main Aug 16, 2025
1 check passed
@bnbarham bnbarham deleted the multi-root-release branch August 16, 2025 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants