Commit c7f9a36c authored by torne's avatar torne Committed by Commit bot

android_webview: Enable FDO support for some targets.

Enable feedback-directed optimisation for the main library (required to
make the FDO symbols present at link time), skia, and sqlite. Update the
android_exports whitelist to allow the gcov symbols to remain public as
otherwise profiling will fail.

BUG=
TBR=michaeln@chromium.org

Review URL: https://codereview.chromium.org/591153002

Cr-Commit-Position: refs/heads/master@{#296997}
parent a0f4c7f7
...@@ -31,6 +31,11 @@ ...@@ -31,6 +31,11 @@
'../ui/android/ui_android.gyp:page_transition_types_java', '../ui/android/ui_android.gyp:page_transition_types_java',
'../ui/android/ui_android.gyp:window_open_disposition_java', '../ui/android/ui_android.gyp:window_open_disposition_java',
], ],
# Enable feedback-directed optimisation for the library when building in
# android.
'aosp_build_settings': {
'LOCAL_FDO_SUPPORT': 'true',
},
}], }],
[ 'android_webview_build==1 and use_system_skia==0', { [ 'android_webview_build==1 and use_system_skia==0', {
# When not using the system skia there are linker warnings about # When not using the system skia there are linker warnings about
......
...@@ -10,5 +10,6 @@ ...@@ -10,5 +10,6 @@
global: global:
Java_*_native*; Java_*_native*;
JNI_OnLoad; JNI_OnLoad;
__gcov_*;
local: *; local: *;
}; };
...@@ -114,6 +114,13 @@ ...@@ -114,6 +114,13 @@
'SK_ARM_HAS_OPTIONAL_NEON', 'SK_ARM_HAS_OPTIONAL_NEON',
], ],
}], }],
# Enable feedback-directed optimisation for skia when building in android.
[ 'android_webview_build == 1', {
'aosp_build_settings': {
'LOCAL_FDO_SUPPORT': 'true',
},
}],
], ],
'variables': { 'variables': {
......
...@@ -161,6 +161,12 @@ ...@@ -161,6 +161,12 @@
'-Wno-pointer-to-int-cast', '-Wno-pointer-to-int-cast',
], ],
}], }],
# Enable feedback-directed optimisation for sqlite when building in android.
['android_webview_build == 1', {
'aosp_build_settings': {
'LOCAL_FDO_SUPPORT': 'true',
},
}],
], ],
}], }],
], ],
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment