Skip to content

Require gold on linux and freebsd by default. #1533

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

Merged

Conversation

gottesmm
Copy link
Contributor

This commit just changes the default to require gold. The user can still override from the command line if they choose to.

The reason why I am making this change is to match Swift's behavior here. There
really isn't any good reason to have separate linkers compiling separate parts
of the same build.

rdar://39456714
(cherry picked from commit 931da81)

This commit just changes the default to require gold. The user can still override from the command line if they choose to.

The reason why I am making this change is to match Swift's behavior here. There
really isn't any good reason to have separate linkers compiling separate parts
of the same build.

rdar://39456714
(cherry picked from commit 931da81)
@gottesmm gottesmm requested review from millenomi and parkera April 24, 2018 20:52
@gottesmm
Copy link
Contributor Author

@swift-ci test

@gottesmm gottesmm requested a review from phausler April 24, 2018 21:43
@millenomi
Copy link
Contributor

@swift-ci please test and merge

@swift-ci swift-ci merged commit b192716 into swiftlang:swift-4.2-branch Apr 24, 2018
@gottesmm gottesmm deleted the swift-4.2-branch-rdar39456714 branch April 24, 2018 22:38
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.

4 participants