diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2022-07-21 15:15:54 +0200 |
|---|---|---|
| committer | Rémi Verschelde <rverschelde@gmail.com> | 2022-09-08 10:00:02 +0200 |
| commit | c2c659db326591519d451d368c4e33c78bb9c1fa (patch) | |
| tree | 3c93bbc143612aa2d5487b9c0918b74629bdd714 /platform/macos | |
| parent | 69233093d7e6479b5130bf2c39cbf464a6809c1b (diff) | |
SCons: Refactor LTO options with `lto=<none|thin|full>`
Adds support for LTO on macOS and Android. We don't have much experience
with LTO on these platforms so for now we keep it disabled by default
even when `production=yes` is set.
Similarly for iOS where we ship object files for the user to link in
Xcode so LTO makes builds extremely slow to link.
`production=yes` defaults to full LTO.
ThinLTO is much faster for LLVM-based compilers but seems to produce
bigger binaries (at least for the Web platform).
Diffstat (limited to 'platform/macos')
| -rw-r--r-- | platform/macos/detect.py | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/platform/macos/detect.py b/platform/macos/detect.py index bd89e8d59f..bcf4776609 100644 --- a/platform/macos/detect.py +++ b/platform/macos/detect.py @@ -40,6 +40,9 @@ def get_flags(): return [ ("arch", detect_arch()), ("use_volk", False), + # Benefits of LTO for macOS (size, performance) haven't been clearly established yet. + # So for now we override the default value which may be set when using `production=yes`. + ("lto", "none"), ] @@ -166,6 +169,15 @@ def configure(env): env["RANLIB"] = basecmd + "ranlib" env["AS"] = basecmd + "as" + # LTO + if env["lto"] != "none": + if env["lto"] == "thin": + env.Append(CCFLAGS=["-flto=thin"]) + env.Append(LINKFLAGS=["-flto=thin"]) + else: + env.Append(CCFLAGS=["-flto"]) + env.Append(LINKFLAGS=["-flto"]) + if env["use_ubsan"] or env["use_asan"] or env["use_tsan"]: env.extra_suffix += ".san" env.Append(CCFLAGS=["-DSANITIZERS_ENABLED"]) |