AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

add clamp_gamut

Browse Source 
This should be cheaper in both float and lowp modes.

Change-Id: If6bba3abd31990b622f010c8fcdebff3288c9d70
Reviewed-on: https://skia-review.googlesource.com/152581
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
This commit is contained in:
Mike Klein 2018-09-07 11:08:53 -04:00 committed by Skia Commit-Bot
parent b5f23f398d
commit eb50f434f0
3 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/core/SkRasterPipeline.h
View File

@ -37,7 +37,7 @@
#define SK_RASTER_PIPELINE_STAGES(M) \
M(callback) \
M(move_src_dst) M(move_dst_src) \
M(clamp_0) M(clamp_1) M(clamp_a) M(clamp_a_dst) \
M(clamp_0) M(clamp_1) M(clamp_a) M(clamp_a_dst) M(clamp_gamut) \
M(unpremul) M(premul) M(premul_dst) \
M(force_opaque) M(force_opaque_dst) \
M(set_rgb) M(swap_rb) \

4
src/core/SkRasterPipelineBlitter.cpp
View File

@ -97,9 +97,7 @@ static void append_color_pipeline(SkRasterPipeline* p,
dstInfo.colorType() != kRGBA_F32_SkColorType &&
dstInfo.alphaType() == kPremul_SkAlphaType)
{
// TODO: this will be common enough that we may want to fuse into ::clamp_premul.
p->append(SkRasterPipeline::clamp_0);
p->append(SkRasterPipeline::clamp_a);
p->append(SkRasterPipeline::clamp_gamut);
}
}

12
src/opts/SkRasterPipeline_opts.h
View File

@ -1276,6 +1276,13 @@ STAGE(clamp_a_dst, Ctx::None) {
db = min(db, da);
}
STAGE(clamp_gamut, Ctx::None) {
// If you're using this stage, a should already be in [0,1].
r = min(max(r, 0), a);
g = min(max(g, 0), a);
b = min(max(b, 0), a);
}
STAGE(set_rgb, const float* rgb) {
r = rgb[0];
g = rgb[1];
@ -2573,6 +2580,11 @@ STAGE_PP(clamp_a_dst, Ctx::None) {
db = min(db, da);
}
STAGE_PP(clamp_gamut, Ctx::None) {
// It shouldn't be possible to get out-of-gamut
// colors when working in lowp.
}
STAGE_PP(premul, Ctx::None) {
r = div255(r * a);
g = div255(g * a);