[3.x] SCons: Refactor Linux linker options with linker=<bfd|gold|lld|mold>
#63283
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
akien-mga
changed the title
linker=<bfd|gold|lld|mold>linker=<bfd|gold|lld|mold>
akien-mga
force-pushed
the
3.x-scons-linux-refactor-linker
branch
from
1c9770d
to
d5b1949
Compare
The new option is `linker` and lets the user specify the argument to the`-fuse_ld=` linker flag directly. The supported options are: - `default`: No change, typically uses GNU ld (bfd) unless the user or distro picked a different default `/usr/bin/ld`. - `bfd`: GNU ld from binutils - `gold`: GNU gold from binutils - `lld`: lld from LLVM - `mold`: mold, an extremely fast modern linker, not (yet) intended for use in production but great for development speed. Provided by distro `mold` package or needs to be compiled from source and installed to `/usr` otherwise. Deprecates the `use_lld=yes` option, and make lld actually usable with GCC too. Not all the above are compatible or recommend for LTO, we recommend using GNU ld with GCC LTO, or lld with LLVM ThinLTO. (cherry picked from commit 534f85add1daec0669a1f18edd2cc456f9a296ef)
akien-mga
force-pushed
the
3.x-scons-linux-refactor-linker
branch
from
d5b1949
to
5bb3063
Compare
|
Ah so that's why my link time shot up this week. 😀 I'm not sure the alias for
|
|
Yeah seems like I used |
akien-mga
added a commit
to akien-mga/godot
that referenced
this pull request
Sep 5, 2022
Adds support for LTO on macOS and Android. Disable LTO by default on iOS even if `production=yes` is set. Also add `linker` option to `server` platform missed in godotengine#63283. Refactor code handling old arguments to make it simpler (breaks compat, but is explicit enough about it and scripts are easy to fix).
Riordan-DC
pushed a commit
to Riordan-DC/godot
that referenced
this pull request
Jan 24, 2023
Adds support for LTO on macOS and Android. Disable LTO by default on iOS even if `production=yes` is set. Also add `linker` option to `server` platform missed in godotengine#63283. Refactor code handling old arguments to make it simpler (breaks compat, but is explicit enough about it and scripts are easy to fix).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #63278, with the difference that I kept a compatibility alias for
use_lld=yes.