From f60e1627edd22f05ab6ba1aab6cc390fa4393a41 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Fri, 4 Aug 2017 09:35:26 -0400 Subject: [PATCH] clamp_1 is also a no-op with 8-bit lowp Change-Id: Ifef97d8f28c88c4ee3f7701aac6e383940ed5275 Reviewed-on: https://skia-review.googlesource.com/31020 Reviewed-by: Florin Malita Commit-Queue: Mike Klein --- src/jumper/SkJumper.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/jumper/SkJumper.cpp b/src/jumper/SkJumper.cpp index 20fbce5625..0d44e8facd 100644 --- a/src/jumper/SkJumper.cpp +++ b/src/jumper/SkJumper.cpp @@ -348,8 +348,9 @@ const SkJumper_Engine& SkRasterPipeline::build_pipeline(void** ip) const { void** reset_point = ip; *--ip = (void*)gLowp.just_return; for (const StageList* st = fStages; st; st = st->prev) { - if (st->stage == SkRasterPipeline::clamp_0) { - continue; // No-op in lowp. + if (st->stage == SkRasterPipeline::clamp_0 || + st->stage == SkRasterPipeline::clamp_1) { + continue; // No-ops in lowp. } if (StageFn* fn = gLowp.stages[st->stage]) { if (st->ctx) {