Make GrGLConfigConversionEffect work for Imagination and some other GPUs.
BUG=372341
R=tomhudson@chromium.org, bsalomon@google.com
Author: changjun.yang@intel.com
Review URL: https://codereview.chromium.org/277323002
diff --git a/src/gpu/effects/GrConfigConversionEffect.cpp b/src/gpu/effects/GrConfigConversionEffect.cpp
index ff18ef2..0e0c9d1 100644
--- a/src/gpu/effects/GrConfigConversionEffect.cpp
+++ b/src/gpu/effects/GrConfigConversionEffect.cpp
@@ -37,6 +37,7 @@
builder->fsCodeAppendf("\t%s = %s.bgra;\n", outputColor, outputColor);
} else {
const char* swiz = fSwapRedAndBlue ? "bgr" : "rgb";
+ const char* outputColorH = "outputColorH";
switch (fPMConversion) {
case GrConfigConversionEffect::kMulByAlpha_RoundUp_PMConversion:
builder->fsCodeAppendf(
@@ -60,6 +61,36 @@
builder->fsCodeAppendf("\t\t%s = %s.a <= 0.0 ? vec4(0,0,0,0) : vec4(floor(%s.%s / %s.a * 255.0) / 255.0, %s.a);\n",
outputColor, outputColor, outputColor, swiz, outputColor, outputColor);
break;
+ case GrConfigConversionEffect::kMulByAlpha_RoundUp_HIGHP_PMConversion:
+ builder->fsCodeAppendf("\thighp vec4 %s;\n", outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColorH, outputColor);
+ builder->fsCodeAppendf(
+ "\t\t%s = vec4(ceil(%s.%s * %s.a * 255.0) / 255.0, %s.a);\n",
+ outputColorH, outputColorH, swiz, outputColorH, outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColor, outputColorH);
+ break;
+ case GrConfigConversionEffect::kMulByAlpha_RoundDown_HIGHP_PMConversion:
+ builder->fsCodeAppendf("\thighp vec4 %s;\n", outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColorH, outputColor);
+ builder->fsCodeAppendf(
+ "\t\t%s = vec4(floor(%s.%s * %s.a * 255.0) / 255.0, %s.a);\n",
+ outputColorH, outputColorH, swiz, outputColorH, outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColor, outputColorH);
+ break;
+ case GrConfigConversionEffect::kDivByAlpha_RoundUp_HIGHP_PMConversion:
+ builder->fsCodeAppendf("\thighp vec4 %s;\n", outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColorH, outputColor);
+ builder->fsCodeAppendf("\t\t%s = %s.a <= 0.0 ? vec4(0,0,0,0) : vec4(ceil(%s.%s / %s.a * 255.0) / 255.0, %s.a);\n",
+ outputColorH, outputColorH, outputColorH, swiz, outputColorH, outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColor, outputColorH);
+ break;
+ case GrConfigConversionEffect::kDivByAlpha_RoundDown_HIGHP_PMConversion:
+ builder->fsCodeAppendf("\thighp vec4 %s;\n", outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColorH, outputColor);
+ builder->fsCodeAppendf("\t\t%s = %s.a <= 0.0 ? vec4(0,0,0,0) : vec4(floor(%s.%s / %s.a * 255.0) / 255.0, %s.a);\n",
+ outputColorH, outputColorH, outputColorH, swiz, outputColorH, outputColorH);
+ builder->fsCodeAppendf("\t\t%s = %s;\n", outputColor, outputColorH);
+ break;
default:
SkFAIL("Unknown conversion op.");
break;
@@ -183,6 +214,8 @@
static const PMConversion kConversionRules[][2] = {
{kDivByAlpha_RoundDown_PMConversion, kMulByAlpha_RoundUp_PMConversion},
{kDivByAlpha_RoundUp_PMConversion, kMulByAlpha_RoundDown_PMConversion},
+ {kDivByAlpha_RoundDown_HIGHP_PMConversion, kMulByAlpha_RoundUp_HIGHP_PMConversion},
+ {kDivByAlpha_RoundUp_HIGHP_PMConversion, kMulByAlpha_RoundDown_HIGHP_PMConversion},
};
GrContext::AutoWideOpenIdentityDraw awoid(context, NULL);
diff --git a/src/gpu/effects/GrConfigConversionEffect.h b/src/gpu/effects/GrConfigConversionEffect.h
index 4416d05..74f717f 100644
--- a/src/gpu/effects/GrConfigConversionEffect.h
+++ b/src/gpu/effects/GrConfigConversionEffect.h
@@ -30,6 +30,10 @@
kMulByAlpha_RoundDown_PMConversion,
kDivByAlpha_RoundUp_PMConversion,
kDivByAlpha_RoundDown_PMConversion,
+ kMulByAlpha_RoundUp_HIGHP_PMConversion,
+ kMulByAlpha_RoundDown_HIGHP_PMConversion,
+ kDivByAlpha_RoundUp_HIGHP_PMConversion,
+ kDivByAlpha_RoundDown_HIGHP_PMConversion,
kPMConversionCnt
};
diff --git a/src/gpu/gl/GrGLShaderBuilder.cpp b/src/gpu/gl/GrGLShaderBuilder.cpp
index 4b7de02..d95cd66 100644
--- a/src/gpu/gl/GrGLShaderBuilder.cpp
+++ b/src/gpu/gl/GrGLShaderBuilder.cpp
@@ -630,7 +630,6 @@
this->fsCodeAppendf("\tvec4 %s;\n", outColorName.c_str());
outColor = outColorName;
-
programEffectsBuilder->emitEffect(stage,
effectKeys[e],
outColor.c_str(),