From 35a15e619161798820b2bd6ff46178c5b7ccebcf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Verschelde?= Date: Tue, 13 Sep 2022 17:01:47 +0200 Subject: SCons: Refactor handling of `production` flag and per-platform LTO defaults Fixup to #63288. See #65583 for the bug report. Co-authored-by: Cyberrebell --- platform/android/detect.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'platform/android') diff --git a/platform/android/detect.py b/platform/android/detect.py index 1d9bcdd932..a31bba745f 100644 --- a/platform/android/detect.py +++ b/platform/android/detect.py @@ -47,9 +47,6 @@ def get_flags(): return [ ("arch", "arm64"), # Default for convenience. ("tools", False), - # Benefits of LTO for Android (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"), ] @@ -136,6 +133,10 @@ def configure(env): env.Append(CPPFLAGS=["-UNDEBUG"]) # LTO + + if env["lto"] == "auto": # LTO benefits for Android (size, performance) haven't been clearly established yet. + env["lto"] = "none" + if env["lto"] != "none": if env["lto"] == "thin": env.Append(CCFLAGS=["-flto=thin"]) -- cgit v1.2.3