From 26ec0293b5e80e00d802108cf6d76c0e7ffa2609 Mon Sep 17 00:00:00 2001 From: Fredrik Fornwall Date: Wed, 19 Sep 2018 23:32:30 +0200 Subject: [PATCH] Try using -Oz on 32-bit arm as well after ndk r18 --- build-package.sh | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/build-package.sh b/build-package.sh index 82c82bd937..3853b35eba 100755 --- a/build-package.sh +++ b/build-package.sh @@ -613,14 +613,7 @@ termux_step_setup_toolchain() { if [ -n "$TERMUX_DEBUG" ]; then CFLAGS+=" -g3 -O1 -fstack-protector --param ssp-buffer-size=4 -D_FORTIFY_SOURCE=2" else - # -Oz seems good for clang, see https://github.com/android-ndk/ndk/issues/133. - # However, on arm it has a lot of issues such as #1520, #1680, #1765 and - # https://bugs.llvm.org/show_bug.cgi?id=35379, so use so use -Os there for now: - if [ $TERMUX_ARCH = arm ]; then - CFLAGS+=" -Os" - else - CFLAGS+=" -Oz" - fi + CFLAGS+=" -Oz" fi export CXXFLAGS="$CFLAGS"