Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit db4113fd authored by Ben Cheng's avatar Ben Cheng Committed by Android Git Automerger
Browse files

am d58df2de: Merge "Re-tighten default compiler options."

* commit 'd58df2de':
  Re-tighten default compiler options.
parents f482fa84 d58df2de
Loading
Loading
Loading
Loading
+5 −6
Original line number Diff line number Diff line
@@ -109,14 +109,13 @@ android_config_h := $(call select-android-config-h,linux-arm)
TARGET_ANDROID_CONFIG_CFLAGS := -include $(android_config_h) -I $(dir $(android_config_h))
TARGET_GLOBAL_CFLAGS += $(TARGET_ANDROID_CONFIG_CFLAGS)

# This warning causes dalvik not to build with gcc 4.6+ and -Werror.
# We cannot turn it off blindly since the option is not available
# in gcc-4.4.x.  We also want to disable sincos optimization globally
# by turning off the builtin sin function.
# The "-Wunused-but-set-variable" option often breaks projects that enable
# "-Wall -Werror" due to a commom idiom "ALOGV(mesg)" where ALOGV is turned
# into no-op in some builds while mesg is defined earlier. So we explicitly
# disable "-Wunused-but-set-variable" here.
ifneq ($(filter 4.6 4.6.% 4.7 4.7.% 4.8, $(TARGET_GCC_VERSION)),)
TARGET_GLOBAL_CFLAGS += -Wno-unused-but-set-variable -fno-builtin-sin \
			-fno-strict-volatile-bitfields \
			-Wno-unused-parameter -Wno-unused-but-set-parameter
			-fno-strict-volatile-bitfields
endif

# This is to avoid the dreaded warning compiler message: