summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJing Yu <jingyu@google.com>2010-12-21 16:33:48 -0800
committerJing Yu <jingyu@google.com>2010-12-21 16:33:48 -0800
commita83b769566550e30b368836e77b5ac3286844768 (patch)
treebdfae0c484e716c81f76ba327fcd3f1643de4479
parent085ff6a8ac90e989507097885eaf65c9587c2c96 (diff)
downloadbuild-a83b769566550e30b368836e77b5ac3286844768.zip
build-a83b769566550e30b368836e77b5ac3286844768.tar.gz
build-a83b769566550e30b368836e77b5ac3286844768.tar.bz2
Remove global inline options.
The purpose of removing global inline options is to give compiler the opportunity to do inline optimizations and inline tunings for Android native codes. By removing these global inline options now, the size of almost all native libraries are reduced. And there is no noticable performance degradation on webkit, gcstone and skia benchmarks. Change-Id: I31e71f51e4f29fa6286fddb89e9eab227581c7b3
-rw-r--r--core/combo/TARGET_linux-arm.mk8
1 files changed, 2 insertions, 6 deletions
diff --git a/core/combo/TARGET_linux-arm.mk b/core/combo/TARGET_linux-arm.mk
index bc398c3..abeaf95 100644
--- a/core/combo/TARGET_linux-arm.mk
+++ b/core/combo/TARGET_linux-arm.mk
@@ -60,8 +60,7 @@ TARGET_NO_UNDEFINED_LDFLAGS := -Wl,--no-undefined
TARGET_arm_CFLAGS := -O2 \
-fomit-frame-pointer \
-fstrict-aliasing \
- -funswitch-loops \
- -finline-limit=300
+ -funswitch-loops
# Modules can choose to compile some source as thumb. As
# non-thumb enabled targets are supported, this is treated
@@ -71,8 +70,7 @@ ifeq ($(ARCH_ARM_HAVE_THUMB_SUPPORT),true)
TARGET_thumb_CFLAGS := -mthumb \
-Os \
-fomit-frame-pointer \
- -fno-strict-aliasing \
- -finline-limit=64
+ -fno-strict-aliasing
else
TARGET_thumb_CFLAGS := $(TARGET_arm_CFLAGS)
endif
@@ -143,8 +141,6 @@ TARGET_RELEASE_CFLAGS := \
-DNDEBUG \
-g \
-Wstrict-aliasing=2 \
- -finline-functions \
- -fno-inline-functions-called-once \
-fgcse-after-reload \
-frerun-cse-after-loop \
-frename-registers