Commit 7dca8e13 authored by Adenilson Cavalcanti's avatar Adenilson Cavalcanti Committed by Commit Bot

Enable compiler optimizations on Harfbuzz

Experiments with hb-shape show that enabling compiler optimizations
improves performance of text shaping (therefore should help layout).

Initial tests show an improvement of 10.4% in hindi-line-layout
and 12.1% in latin-ebook performance layout tests.

Binary-Size: compiler optimized library will increase the size of the apk.
Bug: 902782
Change-Id: Iee50a04403ec634efec17be5ef8a72b996711a2f
Reviewed-on: https://chromium-review.googlesource.com/c/1321250
Commit-Queue: Adenilson Cavalcanti <cavalcantii@chromium.org>
Reviewed-by: default avatarEmil A Eklund <eae@chromium.org>
Reviewed-by: default avatarBehdad Esfahbod <behdad@chromium.org>
Reviewed-by: default avatarDominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606240}
parent ac3ba330
...@@ -205,6 +205,13 @@ if (use_system_harfbuzz) { ...@@ -205,6 +205,13 @@ if (use_system_harfbuzz) {
# correctly. # correctly.
":harfbuzz_warnings", ":harfbuzz_warnings",
] ]
# This allows the compiler to do further optimizations in the code.
if (!is_debug) {
configs -= [ "//build/config/compiler:default_optimization" ]
configs += [ "//build/config/compiler:optimize_speed" ]
}
public_configs = [ ":harfbuzz_config" ] public_configs = [ ":harfbuzz_config" ]
deps = [ deps = [
......
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