Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3670)

Unified Diff: bench/nanobench.cpp

Issue 1775093002: add f16 and srgb --configs to nanobench (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « bench/nanobench.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bench/nanobench.cpp
diff --git a/bench/nanobench.cpp b/bench/nanobench.cpp
index 81bd4f2b5baa145609b61ede1bd168459c26a805..dd792dd39a01299e0c62b543771033a045c647fc 100644
--- a/bench/nanobench.cpp
+++ b/bench/nanobench.cpp
@@ -396,7 +396,7 @@ static void create_config(const SkCommandLineConfig* config, SkTArray<Config>* 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<Config>* 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<Config>* 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;
« no previous file with comments | « bench/nanobench.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698