From: mtklein Date: Tue, 8 Mar 2016 19:31:11 +0000 (-0800) Subject: add f16 and srgb --configs to nanobench X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~129^2~1586 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bb6c41b1dbe5021da5c7e8257e04d8ed6979af05;p=platform%2Fupstream%2FlibSkiaSharp.git add f16 and srgb --configs to nanobench BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1775093002 Review URL: https://codereview.chromium.org/1775093002 --- diff --git a/bench/nanobench.cpp b/bench/nanobench.cpp index 81bd4f2..dd792dd 100644 --- a/bench/nanobench.cpp +++ b/bench/nanobench.cpp @@ -396,7 +396,7 @@ static void create_config(const SkCommandLineConfig* config, SkTArray* c if (!FLAGS_gpu) return; - const auto ctxOptions = gpuConfig->getUseNVPR() ? GrContextFactory::kEnableNVPR_GLContextOptions + const auto ctxOptions = gpuConfig->getUseNVPR() ? GrContextFactory::kEnableNVPR_GLContextOptions : GrContextFactory::kNone_GLContextOptions; const auto ctxType = gpuConfig->getContextType(); const auto sampleCount = gpuConfig->getSamples(); @@ -418,6 +418,7 @@ static void create_config(const SkCommandLineConfig* config, SkTArray* c Benchmark::kGPU_Backend, kN32_SkColorType, kPremul_SkAlphaType, + kLinear_SkColorProfileType, sampleCount, ctxType, ctxOptions, @@ -429,28 +430,37 @@ static void create_config(const SkCommandLineConfig* config, SkTArray* c } #endif - #define CPU_CONFIG(name, backend, color, alpha) \ - if (config->getTag().equals(#name)) { \ - Config config = { SkString(#name), Benchmark::backend, color, alpha, 0, \ - kBogusGLContextType, kBogusGLContextOptions, \ - false }; \ - configs->push_back(config); \ - return; \ + #define CPU_CONFIG(name, backend, color, alpha, profile) \ + if (config->getTag().equals(#name)) { \ + Config config = { \ + SkString(#name), Benchmark::backend, color, alpha, profile, \ + 0, kBogusGLContextType, kBogusGLContextOptions, false \ + }; \ + configs->push_back(config); \ + return; \ } if (FLAGS_cpu) { - CPU_CONFIG(nonrendering, kNonRendering_Backend, kUnknown_SkColorType, kUnpremul_SkAlphaType) - CPU_CONFIG(8888, kRaster_Backend, kN32_SkColorType, kPremul_SkAlphaType) - CPU_CONFIG(565, kRaster_Backend, kRGB_565_SkColorType, kOpaque_SkAlphaType) + CPU_CONFIG(nonrendering, kNonRendering_Backend, + kUnknown_SkColorType, kUnpremul_SkAlphaType, kLinear_SkColorProfileType); + + CPU_CONFIG(8888, kRaster_Backend, + kN32_SkColorType, kPremul_SkAlphaType, kLinear_SkColorProfileType) + CPU_CONFIG(565, kRaster_Backend, + kRGB_565_SkColorType, kOpaque_SkAlphaType, kLinear_SkColorProfileType) + CPU_CONFIG(srgb, kRaster_Backend, + kN32_SkColorType, kPremul_SkAlphaType, kSRGB_SkColorProfileType) + CPU_CONFIG(f16, kRaster_Backend, + kRGBA_F16_SkColorType, kPremul_SkAlphaType, kLinear_SkColorProfileType) } #undef CPU_CONFIG #ifdef SK_BUILD_FOR_ANDROID_FRAMEWORK if (config->getTag().equals("hwui")) { - Config config = { SkString("hwui"), Benchmark::kHWUI_Backend, kRGBA_8888_SkColorType, - kPremul_SkAlphaType, 0, kBogusGLContextType, kBogusGLContextOptions, - false }; + Config config = { SkString("hwui"), Benchmark::kHWUI_Backend, + kRGBA_8888_SkColorType, kPremul_SkAlphaType, kLinear_SkColorProfileType, + 0, kBogusGLContextType, kBogusGLContextOptions, false }; configs->push_back(config); } #endif @@ -472,7 +482,7 @@ static Target* is_enabled(Benchmark* bench, const Config& config) { } SkImageInfo info = SkImageInfo::Make(bench->getSize().fX, bench->getSize().fY, - config.color, config.alpha); + config.color, config.alpha, config.profile); Target* target = nullptr; diff --git a/bench/nanobench.h b/bench/nanobench.h index 204ada6..ad538dd 100644 --- a/bench/nanobench.h +++ b/bench/nanobench.h @@ -26,6 +26,7 @@ struct Config { Benchmark::Backend backend; SkColorType color; SkAlphaType alpha; + SkColorProfileType profile; int samples; #if SK_SUPPORT_GPU GrContextFactory::GLContextType ctxType;