From 5516927b9231f622e1db0606ea876df4cefb9ce6 Mon Sep 17 00:00:00 2001 From: bxu10X Date: Tue, 6 Mar 2012 15:05:11 +0800 Subject: V8: 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 external/webkit Change-Id: Ib055e70c2e0587ac66dcc29ccef8e455aa0b832e Author: Bin Xu 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.libv8.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Android.libv8.mk b/Android.libv8.mk index cd32be65..35e5e5f1 100644 --- a/Android.libv8.mk +++ b/Android.libv8.mk @@ -104,7 +104,7 @@ ifeq ($(TARGET_ARCH),mips) endif ifeq ($(TARGET_ARCH),x86) - LOCAL_CFLAGS += -DV8_TARGET_ARCH_IA32 + LOCAL_CFLAGS += -DV8_TARGET_ARCH_IA32 -fno-pic endif ifeq ($(DEBUG_V8),true) -- cgit v1.2.3