diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2022-09-23 13:55:00 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2022-09-23 13:56:16 +0200 |
commit | c5c3d13dc0dbbf95a2350f57d69e7cdc546d395d (patch) | |
tree | 81048740b043f28a55aeba6c03e355adde098145 /platform/android | |
parent | 006e345695a0448acefd63dcb726a07b63cb1d0d (diff) |
SCons: Remove redundant `-fomit-frame-pointer` and `-ftree-vectorize`
- `-fomit-frame-pointer` is included automatically by both GCC and
Clang in `-O1` and above.
- `-ftree-vectorize` is included automatically by GCC in `-O2` and
beyond, and seems always enabled by Clang.
Closes #66296. See that issue for a detailed investigation.
Diffstat (limited to 'platform/android')
-rw-r--r-- | platform/android/detect.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/platform/android/detect.py b/platform/android/detect.py index a31bba745f..f36e7e2a7f 100644 --- a/platform/android/detect.py +++ b/platform/android/detect.py @@ -121,11 +121,10 @@ def configure(env): # `-O2` is more friendly to debuggers than `-O3`, leading to better crash backtraces # when using `target=release_debug`. opt = "-O3" if env["target"] == "release" else "-O2" - env.Append(CCFLAGS=[opt, "-fomit-frame-pointer"]) + env.Append(CCFLAGS=[opt]) elif env["optimize"] == "size": # optimize for size env.Append(CCFLAGS=["-Oz"]) env.Append(CPPDEFINES=["NDEBUG"]) - env.Append(CCFLAGS=["-ftree-vectorize"]) elif env["target"] == "debug": env.Append(LINKFLAGS=["-O0"]) env.Append(CCFLAGS=["-O0", "-g", "-fno-limit-debug-info"]) |