Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2c0f9379 authored by John Reck's avatar John Reck
Browse files

Add a font microbench 

Also remove a low-signal high-frequency trace mark

Fixes: 27894187

Change-Id: I090122a0db8e455eaa9dbce2b0f5b6778a218188
parent a89a392a

libs/hwui/Android.mk

+1 −0
Original line number Diff line number Diff line
@@ -322,6 +322,7 @@ LOCAL_SRC_FILES += \ 
    $(hwui_test_common_src_files) \

    tests/microbench/main.cpp \

    tests/microbench/DisplayListCanvasBench.cpp \

    tests/microbench/FontBench.cpp \

    tests/microbench/LinearAllocatorBench.cpp \

    tests/microbench/PathParserBench.cpp \

    tests/microbench/ShadowBench.cpp \

libs/hwui/font/Font.cpp

+0 −2
Original line number Diff line number Diff line
@@ -356,8 +356,6 @@ void Font::measure(const SkPaint* paint, const glyph_t* glyphs, 
}



void Font::precache(const SkPaint* paint, const glyph_t* glyphs, int numGlyphs) {

    ATRACE_NAME("Precache Glyphs");



    if (numGlyphs == 0 || glyphs == nullptr) {

        return;

    }

libs/hwui/tests/microbench/FontBench.cpp

0 → 100644
+50 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2016 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



#include <benchmark/benchmark.h>



#include "GammaFontRenderer.h"

#include "tests/common/TestUtils.h"



#include <SkPaint.h>



using namespace android;

using namespace android::uirenderer;



void BM_FontRenderer_precache_cachehits(benchmark::State& state) {

    TestUtils::runOnRenderThread([&state](renderthread::RenderThread& thread) {

        SkPaint paint;

        paint.setTextSize(20);

        paint.setTextEncoding(SkPaint::kGlyphID_TextEncoding);

        GammaFontRenderer gammaFontRenderer;

        FontRenderer& fontRenderer = gammaFontRenderer.getFontRenderer();

        fontRenderer.setFont(&paint, SkMatrix::I());



        std::vector<glyph_t> glyphs;

        std::vector<float> positions;

        float totalAdvance;

        uirenderer::Rect bounds;

        TestUtils::layoutTextUnscaled(paint, "This is a test",

                &glyphs, &positions, &totalAdvance, &bounds);



        fontRenderer.precache(&paint, glyphs.data(), glyphs.size(), SkMatrix::I());



        while (state.KeepRunning()) {

            fontRenderer.precache(&paint, glyphs.data(), glyphs.size(), SkMatrix::I());

        }

    });

}

BENCHMARK(BM_FontRenderer_precache_cachehits);

libs/hwui/tests/microbench/FrameBuilderBench.cpp

+11 −9
Original line number Diff line number Diff line
@@ -113,6 +113,7 @@ static auto SCENES = { 
};



void BM_FrameBuilder_defer_scene(benchmark::State& state) {

    TestUtils::runOnRenderThread([&state](RenderThread& thread) {

        const char* sceneName = *(SCENES.begin() + state.range_x());

        state.SetLabel(sceneName);

        auto nodes = getSyncedSceneNodes(sceneName);
@@ -122,6 +123,7 @@ void BM_FrameBuilder_defer_scene(benchmark::State& state) { 
                    nodes, sLightGeometry, Caches::getInstance());

            benchmark::DoNotOptimize(&frameBuilder);

        }

    });

}

BENCHMARK(BM_FrameBuilder_defer_scene)->DenseRange(0, SCENES.size() - 1);