Skip to content

[5.6 CMake] Move most Standard Library options in a separate file #40669

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

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented Dec 21, 2021

This allows the file to be easily included where needed (e.g.
StandaloneOverlay.cmake) and reduce the likelihood of miscompilation
due to missing sensible defaults.

Focus on parameters only in stdlib/CMakeLists.txt whose defaults are not
computed.

This matches #40610 + #40640

Addresses rdar://86740965

This allows the file to be easily included where needed (e.g.
`StandaloneOverlay.cmake`) and reduce the likelihood of miscompilation
due to missing sensible defaults.

Focus on parameters only in stdlib/CMakeLists.txt whose defaults are not
computed.

This matches swiftlang#40610 + swiftlang#40640

Addresses rdar://86740965
@edymtt
Copy link
Contributor Author

edymtt commented Dec 21, 2021

@swift-ci please test Apple Silicon

@edymtt
Copy link
Contributor Author

edymtt commented Dec 21, 2021

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented Dec 21, 2021

@swift-ci please build toolchain

@swift-ci
Copy link
Contributor

Linux Toolchain (Ubuntu 16.04)
Download Toolchain
Git Sha - 49592c9

Install command
tar zxf swift-PR-40669-759-ubuntu16.04.tar.gz
More info

@swift-ci
Copy link
Contributor

macOS Toolchain
Download Toolchain
Git Sha - 49592c9

Install command
tar -zxf swift-PR-40669-1254-osx.tar.gz --directory ~/

Copy link
Contributor

@gottesmm gottesmm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me.

@edymtt edymtt merged commit 0bf73bb into swiftlang:release/5.6 Jan 4, 2022
@edymtt edymtt deleted the move-most-stdlib-options-to-separate-file-5.6 branch January 10, 2022 17:21
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