From a83b769566550e30b368836e77b5ac3286844768 Mon Sep 17 00:00:00 2001 From: Jing Yu Date: Tue, 21 Dec 2010 16:33:48 -0800 Subject: 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 --- core/combo/TARGET_linux-arm.mk | 8 ++------ 1 file 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 -- cgit v1.1