From 33e9f1a70d5f58639b524f40bf39a8e233c04ba8 Mon Sep 17 00:00:00 2001 From: Nicolas Geoffray Date: Mon, 29 Jan 2024 14:24:31 +0000 Subject: Reland^2 "Run optimizations with baseline compilation." This reverts commit 3dccb13f4e92db37a13359e126c5ddc12cb674b5. Also includes the fix for incrementing hotness that got reverted: aosp/2906378 Bug: 313040662 Reduces jank on compose view scrolling for 4 iterations: - For Go Mokey: - Before: ~698 frames drawn / ~13.87% janky frames - After: ~937 frames drawn / ~5.52% janky frames - For Pixel 8 pro: - Before: ~2440 frames drawn / ~0.90% janky frames - After: ~2450 frames drawn / ~0.55% janky frames Reason for revert: Reduce inlining threshold for baseline. Change-Id: Iee5cd4c3ceb7715caf9299b56551aae6f0259769 --- compiler/driver/compiler_options.h | 2 ++ 1 file changed, 2 insertions(+) (limited to 'compiler/driver/compiler_options.h') diff --git a/compiler/driver/compiler_options.h b/compiler/driver/compiler_options.h index 5890227c36..f659b12959 100644 --- a/compiler/driver/compiler_options.h +++ b/compiler/driver/compiler_options.h @@ -65,6 +65,8 @@ class CompilerOptions final { static const bool kDefaultGenerateDebugInfo = false; static const bool kDefaultGenerateMiniDebugInfo = true; static const size_t kDefaultInlineMaxCodeUnits = 32; + // We set a lower inlining threshold for baseline to reduce code size and compilation time. + static const size_t kBaselineMaxCodeUnits = 8; static constexpr size_t kUnsetInlineMaxCodeUnits = -1; enum class CompilerType : uint8_t { -- cgit v1.2.3-59-g8ed1b