From 068ccaa063469dd16cae646e30b0eecb8bb5065a Mon Sep 17 00:00:00 2001 From: Pirama Arumuga Nainar Date: Mon, 5 Mar 2018 10:20:38 -0800 Subject: Re-enable PGO for Skia Also use the same profile file as Android's hwui (hwui/hwui.profdata). Since hwui and skia both also use LTO, the profile files need to be the same to satisfy the Clang LTO plugin. Test: Verify that the miscompiles are fixed. Change-Id: I2a5217b64966144319e63b83351dea2388b60250 Reviewed-on: https://skia-review.googlesource.com/112106 Reviewed-by: Zhizhou Yang Reviewed-by: Derek Sollenberger Commit-Queue: Derek Sollenberger --- gn/gn_to_bp.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'gn/gn_to_bp.py') diff --git a/gn/gn_to_bp.py b/gn/gn_to_bp.py index 4c8cba75d5..d8ec9e304a 100644 --- a/gn/gn_to_bp.py +++ b/gn/gn_to_bp.py @@ -107,11 +107,9 @@ cc_defaults { name: "skia_pgo", pgo: { instrumentation: true, - profile_file: "skia/skia.profdata", + profile_file: "hwui/hwui.profdata", benchmarks: ["hwui", "skia"], - // Bugs: http://b/73127367, http://b/73257154, http://b/73249590 - // Turn off PGO for Skia until toolchain bug is fixed. - enable_profile_use: false, + enable_profile_use: true, }, } -- cgit v1.2.3