Skip to content

Build Swift-Testing with WMO #81788

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.

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

Open
wants to merge 1 commit into
base: release/6.2
Choose a base branch
from

Conversation

etcwilde
Copy link
Contributor

This tells build-script to build Swift-testing with WMO. This results in a faster build products, but is also necessary for configurations using the legacy swift driver, which would emit an invalid swift interface in non-WMO builds.

  • Explanation: The merge module job in the old Swift driver emits a broken textual swift interface resulting in a broken bootstrap build (Testing.swiftinterface:413:2: error: '@_optimize(none)' attribute cannot be applied to stored properties). Building swift-testing with WMO enabled results in a faster library and does not involve the broken merge-modules job.
  • Risk: Low. This is enabling WMO.
  • Original PR: Build Swift-Testing with WMO #81622
  • Reviewed by: @stmontgomery
  • Resolves: rdar://151357567
  • Tests: PR testing, nightly toolchain, mainline bootstrap builds

This tells build-script to build Swift-testing with WMO.
This results in a faster build products, but is also necessary for
configurations using the legacy swift driver, which would emit an
invalid swift interface in non-WMO builds.

Fixes: rdar://151357567
(cherry picked from commit 5f2b002)
@etcwilde etcwilde requested a review from a team as a code owner May 27, 2025 21:51
@etcwilde etcwilde added swift 6.2 🍒 release cherry pick Flag: Release branch cherry picks labels May 27, 2025
@etcwilde
Copy link
Contributor Author

@swift-ci please test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 6.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants