From 3c01f41e1ae6b1aa0bd1fb6e37bc02fa2ca3c62e Mon Sep 17 00:00:00 2001 From: bxu10X Date: Tue, 6 Mar 2012 13:08:06 +0800 Subject: webkit: enable no-pic compiling option This patch is to enable no-pic compiling option PIC enables efficient code sharing between processes but increases overhead due to indirectly access global variables and functions. By enabling no-pic here, we can get rid of indirect access calling __x86.get_pc_thunk.bx, while still share the library between processes, since zygote loads libwebcore, and all android applications are forked from zygote and inherits its address space. The patch takes effect together with the one enabling no-pic compiling option in v8. With them, we can see obvious performance improvement for browser workloads, 5%+ for HTML5-zoom and HTML5-animation, 6.6% for sunspider, and 9.1% for pageloading Note: This patch is related to a change in platform/external/v8 Change-Id: I47ee3149d31b2f7d78c4a399503b6d9a9f0f572a Author: Lijuan Xiao Signed-off-by: Bin Xu Singed-off-by: Shuo Gao Signed-off-by: Bruce Beare Signed-off-by: Jack Ren Author-tracking-BZ: 26358 --- Android.mk | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'Android.mk') diff --git a/Android.mk b/Android.mk index 61bd467..12fbb59 100644 --- a/Android.mk +++ b/Android.mk @@ -252,6 +252,10 @@ LOCAL_CFLAGS += -Darm LOCAL_CFLAGS += -Wno-psabi endif +ifeq ($(TARGET_ARCH_VARIANT),x86-atom) +LOCAL_CFLAGS += -fno-pic +endif + # need a flag to tell the C side when we're on devices with large memory # budgets (i.e. larger than the low-end devices that initially shipped) ifeq ($(ARCH_ARM_HAVE_VFP),true) -- cgit v1.1