Skip to content

Conversation

al45tair
Copy link
Contributor

@al45tair al45tair commented Oct 6, 2025

We don't want library evolution turned on for the Static SDK.

We can't do this in the StdlibOptions.cmake file because that's testing the host, not the target (the code in there is wrong).

rdar://160455139

We don't want library evolution turned on for the Static SDK.

We can't do this in the `StdlibOptions.cmake` file because that's
testing the *host*, not the *target* (the code in there is wrong).

rdar://160455139
@al45tair al45tair requested a review from etcwilde October 6, 2025 09:42
@al45tair al45tair requested a review from a team as a code owner October 6, 2025 09:42
@al45tair al45tair added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels Oct 6, 2025
@al45tair
Copy link
Contributor Author

al45tair commented Oct 6, 2025

@swift-ci Please test

@al45tair
Copy link
Contributor Author

al45tair commented Oct 6, 2025

Explanation: Disables library evolution for the Static SDK build. Has no effect on any other builds.
Risk: Low. Only affects the Static SDK, and that doesn't need library evolution enabled.
Original PR: #84705
Reviewed by: TBC (but hopefully @etcwilde)
Resolves: rdar://160455139
Tests: There are no tests for this. It doesn't matter if library evolution is enabled or not, except in as much as enabling it reduces optimisation opportunities for the compiler and prevents us from getting the smallest possible binaries.

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.

1 participant