Remove the last use of fragmentProcessor field access in .fp files
This is a complex feature that was propping up this one use-case. After
this CL, it can be removed.
Change-Id: Ia391565daddeba46adfb325980d10ad653e269c8
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/375019
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: John Stiles <johnstiles@google.com>
diff --git a/src/gpu/gradients/GrClampedGradientEffect.fp b/src/gpu/gradients/GrClampedGradientEffect.fp
index 2e9a513..53b7de8 100644
--- a/src/gpu/gradients/GrClampedGradientEffect.fp
+++ b/src/gpu/gradients/GrClampedGradientEffect.fp
@@ -23,6 +23,7 @@
layout(key) in bool makePremul;
// Trust the creator that this matches the color spec of the gradient
in bool colorsAreOpaque;
+layout(key) in bool layoutPreservesOpacity;
half4 main() {
half4 t = sample(gradLayout);
@@ -31,7 +32,7 @@
// If t.x is below 0, use the left border color without invoking the child processor. If any t.x
// is above 1, use the right border color. Otherwise, t is in the [0, 1] range assumed by the
// colorizer FP, so delegate to the child processor.
- if (!gradLayout.preservesOpaqueInput && t.y < 0) {
+ if (!layoutPreservesOpacity && t.y < 0) {
// layout has rejected this fragment (rely on sksl to remove this branch if the layout FP
// preserves opacity is false)
outColor = half4(0);
@@ -56,6 +57,21 @@
// into account the opacity of the border colors).
@optimizationFlags {
kCompatibleWithCoverageAsAlpha_OptimizationFlag |
- (colorsAreOpaque && gradLayout->preservesOpaqueInput() ? kPreservesOpaqueInput_OptimizationFlag
- : kNone_OptimizationFlags)
+ (colorsAreOpaque && layoutPreservesOpacity ? kPreservesOpaqueInput_OptimizationFlag
+ : kNone_OptimizationFlags)
+}
+
+@make{
+ static std::unique_ptr<GrFragmentProcessor> Make(
+ std::unique_ptr<GrFragmentProcessor> colorizer,
+ std::unique_ptr<GrFragmentProcessor> gradLayout,
+ SkPMColor4f leftBorderColor,
+ SkPMColor4f rightBorderColor,
+ bool makePremul,
+ bool colorsAreOpaque) {
+ bool layoutPreservesOpacity = gradLayout->preservesOpaqueInput();
+ return std::unique_ptr<GrFragmentProcessor>(new GrClampedGradientEffect(
+ std::move(colorizer), std::move(gradLayout), leftBorderColor, rightBorderColor,
+ makePremul, colorsAreOpaque, layoutPreservesOpacity));
+ }
}
diff --git a/src/gpu/gradients/GrTiledGradientEffect.fp b/src/gpu/gradients/GrTiledGradientEffect.fp
index 30b94ff..3a2cd97 100644
--- a/src/gpu/gradients/GrTiledGradientEffect.fp
+++ b/src/gpu/gradients/GrTiledGradientEffect.fp
@@ -14,11 +14,12 @@
layout(key) in bool makePremul;
// Trust the creator that this matches the color spec of the gradient
in bool colorsAreOpaque;
+layout(key) in bool layoutPreservesOpacity;
half4 main() {
half4 t = sample(gradLayout);
- if (!gradLayout.preservesOpaqueInput && t.y < 0) {
+ if (!layoutPreservesOpacity && t.y < 0) {
// layout has rejected this fragment (rely on sksl to remove this branch if the layout FP
// preserves opacity is false)
return half4(0);
@@ -54,6 +55,20 @@
// but otherwise respect the provided color opacity state.
@optimizationFlags {
kCompatibleWithCoverageAsAlpha_OptimizationFlag |
- (colorsAreOpaque && gradLayout->preservesOpaqueInput() ? kPreservesOpaqueInput_OptimizationFlag
- : kNone_OptimizationFlags)
+ (colorsAreOpaque && layoutPreservesOpacity ? kPreservesOpaqueInput_OptimizationFlag
+ : kNone_OptimizationFlags)
+}
+
+@make{
+ static std::unique_ptr<GrFragmentProcessor> Make(
+ std::unique_ptr<GrFragmentProcessor> colorizer,
+ std::unique_ptr<GrFragmentProcessor> gradLayout,
+ bool mirror,
+ bool makePremul,
+ bool colorsAreOpaque) {
+ bool layoutPreservesOpacity = gradLayout->preservesOpaqueInput();
+ return std::unique_ptr<GrFragmentProcessor>(new GrTiledGradientEffect(
+ std::move(colorizer), std::move(gradLayout), mirror, makePremul, colorsAreOpaque,
+ layoutPreservesOpacity));
+ }
}
diff --git a/src/gpu/gradients/generated/GrClampedGradientEffect.cpp b/src/gpu/gradients/generated/GrClampedGradientEffect.cpp
index 872513c..bc4015d 100644
--- a/src/gpu/gradients/generated/GrClampedGradientEffect.cpp
+++ b/src/gpu/gradients/generated/GrClampedGradientEffect.cpp
@@ -32,6 +32,8 @@
(void)makePremul;
auto colorsAreOpaque = _outer.colorsAreOpaque;
(void)colorsAreOpaque;
+ auto layoutPreservesOpacity = _outer.layoutPreservesOpacity;
+ (void)layoutPreservesOpacity;
leftBorderColorVar = args.fUniformHandler->addUniform(&_outer, kFragment_GrShaderFlag,
kHalf4_GrSLType, "leftBorderColor");
rightBorderColorVar = args.fUniformHandler->addUniform(&_outer, kFragment_GrShaderFlag,
@@ -47,8 +49,7 @@
} else if (t.x > 1.0) {
outColor = %s;
} else {)SkSL",
- _sample0.c_str(),
- (_outer.childProcessor(1)->preservesOpaqueInput() ? "true" : "false"),
+ _sample0.c_str(), (_outer.layoutPreservesOpacity ? "true" : "false"),
args.fUniformHandler->getUniformCStr(leftBorderColorVar),
args.fUniformHandler->getUniformCStr(rightBorderColorVar));
SkString _coords1("float2(half2(t.x, 0.0))");
@@ -93,6 +94,7 @@
void GrClampedGradientEffect::onGetGLSLProcessorKey(const GrShaderCaps& caps,
GrProcessorKeyBuilder* b) const {
b->add32((uint32_t)makePremul);
+ b->add32((uint32_t)layoutPreservesOpacity);
}
bool GrClampedGradientEffect::onIsEqual(const GrFragmentProcessor& other) const {
const GrClampedGradientEffect& that = other.cast<GrClampedGradientEffect>();
@@ -101,6 +103,7 @@
if (rightBorderColor != that.rightBorderColor) return false;
if (makePremul != that.makePremul) return false;
if (colorsAreOpaque != that.colorsAreOpaque) return false;
+ if (layoutPreservesOpacity != that.layoutPreservesOpacity) return false;
return true;
}
GrClampedGradientEffect::GrClampedGradientEffect(const GrClampedGradientEffect& src)
@@ -108,7 +111,8 @@
, leftBorderColor(src.leftBorderColor)
, rightBorderColor(src.rightBorderColor)
, makePremul(src.makePremul)
- , colorsAreOpaque(src.colorsAreOpaque) {
+ , colorsAreOpaque(src.colorsAreOpaque)
+ , layoutPreservesOpacity(src.layoutPreservesOpacity) {
this->cloneAndRegisterAllChildProcessors(src);
}
std::unique_ptr<GrFragmentProcessor> GrClampedGradientEffect::clone() const {
@@ -118,9 +122,10 @@
SkString GrClampedGradientEffect::onDumpInfo() const {
return SkStringPrintf(
"(leftBorderColor=half4(%f, %f, %f, %f), rightBorderColor=half4(%f, %f, %f, %f), "
- "makePremul=%s, colorsAreOpaque=%s)",
+ "makePremul=%s, colorsAreOpaque=%s, layoutPreservesOpacity=%s)",
leftBorderColor.fR, leftBorderColor.fG, leftBorderColor.fB, leftBorderColor.fA,
rightBorderColor.fR, rightBorderColor.fG, rightBorderColor.fB, rightBorderColor.fA,
- (makePremul ? "true" : "false"), (colorsAreOpaque ? "true" : "false"));
+ (makePremul ? "true" : "false"), (colorsAreOpaque ? "true" : "false"),
+ (layoutPreservesOpacity ? "true" : "false"));
}
#endif
diff --git a/src/gpu/gradients/generated/GrClampedGradientEffect.h b/src/gpu/gradients/generated/GrClampedGradientEffect.h
index 5beef16..b67ff40 100644
--- a/src/gpu/gradients/generated/GrClampedGradientEffect.h
+++ b/src/gpu/gradients/generated/GrClampedGradientEffect.h
@@ -25,9 +25,10 @@
SkPMColor4f rightBorderColor,
bool makePremul,
bool colorsAreOpaque) {
+ bool layoutPreservesOpacity = gradLayout->preservesOpaqueInput();
return std::unique_ptr<GrFragmentProcessor>(new GrClampedGradientEffect(
std::move(colorizer), std::move(gradLayout), leftBorderColor, rightBorderColor,
- makePremul, colorsAreOpaque));
+ makePremul, colorsAreOpaque, layoutPreservesOpacity));
}
GrClampedGradientEffect(const GrClampedGradientEffect& src);
std::unique_ptr<GrFragmentProcessor> clone() const override;
@@ -36,6 +37,7 @@
SkPMColor4f rightBorderColor;
bool makePremul;
bool colorsAreOpaque;
+ bool layoutPreservesOpacity;
private:
GrClampedGradientEffect(std::unique_ptr<GrFragmentProcessor> colorizer,
@@ -43,16 +45,18 @@
SkPMColor4f leftBorderColor,
SkPMColor4f rightBorderColor,
bool makePremul,
- bool colorsAreOpaque)
+ bool colorsAreOpaque,
+ bool layoutPreservesOpacity)
: INHERITED(kGrClampedGradientEffect_ClassID,
(OptimizationFlags)kCompatibleWithCoverageAsAlpha_OptimizationFlag |
- (colorsAreOpaque && gradLayout->preservesOpaqueInput()
+ (colorsAreOpaque && layoutPreservesOpacity
? kPreservesOpaqueInput_OptimizationFlag
: kNone_OptimizationFlags))
, leftBorderColor(leftBorderColor)
, rightBorderColor(rightBorderColor)
, makePremul(makePremul)
- , colorsAreOpaque(colorsAreOpaque) {
+ , colorsAreOpaque(colorsAreOpaque)
+ , layoutPreservesOpacity(layoutPreservesOpacity) {
this->registerChild(std::move(colorizer), SkSL::SampleUsage::Explicit());
this->registerChild(std::move(gradLayout), SkSL::SampleUsage::PassThrough());
}
diff --git a/src/gpu/gradients/generated/GrTiledGradientEffect.cpp b/src/gpu/gradients/generated/GrTiledGradientEffect.cpp
index 21700c2..876ff5e 100644
--- a/src/gpu/gradients/generated/GrTiledGradientEffect.cpp
+++ b/src/gpu/gradients/generated/GrTiledGradientEffect.cpp
@@ -30,6 +30,8 @@
(void)makePremul;
auto colorsAreOpaque = _outer.colorsAreOpaque;
(void)colorsAreOpaque;
+ auto layoutPreservesOpacity = _outer.layoutPreservesOpacity;
+ (void)layoutPreservesOpacity;
SkString _sample0 = this->invokeChild(1, args);
fragBuilder->codeAppendf(
R"SkSL(half4 t = %s;
@@ -47,8 +49,7 @@
t.x = fract(t.x);
}
@if (!%s) {)SkSL",
- _sample0.c_str(),
- (_outer.childProcessor(1)->preservesOpaqueInput() ? "true" : "false"),
+ _sample0.c_str(), (_outer.layoutPreservesOpacity ? "true" : "false"),
(_outer.mirror ? "true" : "false"), (_outer.makePremul ? "true" : "false"));
SkString _coords1("float2(half2(t.x, 0.0))");
SkString _sample1 = this->invokeChild(0, args, _coords1.c_str());
@@ -80,6 +81,7 @@
GrProcessorKeyBuilder* b) const {
b->add32((uint32_t)mirror);
b->add32((uint32_t)makePremul);
+ b->add32((uint32_t)layoutPreservesOpacity);
}
bool GrTiledGradientEffect::onIsEqual(const GrFragmentProcessor& other) const {
const GrTiledGradientEffect& that = other.cast<GrTiledGradientEffect>();
@@ -87,13 +89,15 @@
if (mirror != that.mirror) return false;
if (makePremul != that.makePremul) return false;
if (colorsAreOpaque != that.colorsAreOpaque) return false;
+ if (layoutPreservesOpacity != that.layoutPreservesOpacity) return false;
return true;
}
GrTiledGradientEffect::GrTiledGradientEffect(const GrTiledGradientEffect& src)
: INHERITED(kGrTiledGradientEffect_ClassID, src.optimizationFlags())
, mirror(src.mirror)
, makePremul(src.makePremul)
- , colorsAreOpaque(src.colorsAreOpaque) {
+ , colorsAreOpaque(src.colorsAreOpaque)
+ , layoutPreservesOpacity(src.layoutPreservesOpacity) {
this->cloneAndRegisterAllChildProcessors(src);
}
std::unique_ptr<GrFragmentProcessor> GrTiledGradientEffect::clone() const {
@@ -101,8 +105,9 @@
}
#if GR_TEST_UTILS
SkString GrTiledGradientEffect::onDumpInfo() const {
- return SkStringPrintf("(mirror=%s, makePremul=%s, colorsAreOpaque=%s)",
- (mirror ? "true" : "false"), (makePremul ? "true" : "false"),
- (colorsAreOpaque ? "true" : "false"));
+ return SkStringPrintf(
+ "(mirror=%s, makePremul=%s, colorsAreOpaque=%s, layoutPreservesOpacity=%s)",
+ (mirror ? "true" : "false"), (makePremul ? "true" : "false"),
+ (colorsAreOpaque ? "true" : "false"), (layoutPreservesOpacity ? "true" : "false"));
}
#endif
diff --git a/src/gpu/gradients/generated/GrTiledGradientEffect.h b/src/gpu/gradients/generated/GrTiledGradientEffect.h
index 2edad86..101c8b4 100644
--- a/src/gpu/gradients/generated/GrTiledGradientEffect.h
+++ b/src/gpu/gradients/generated/GrTiledGradientEffect.h
@@ -24,8 +24,10 @@
bool mirror,
bool makePremul,
bool colorsAreOpaque) {
- return std::unique_ptr<GrFragmentProcessor>(new GrTiledGradientEffect(
- std::move(colorizer), std::move(gradLayout), mirror, makePremul, colorsAreOpaque));
+ bool layoutPreservesOpacity = gradLayout->preservesOpaqueInput();
+ return std::unique_ptr<GrFragmentProcessor>(
+ new GrTiledGradientEffect(std::move(colorizer), std::move(gradLayout), mirror,
+ makePremul, colorsAreOpaque, layoutPreservesOpacity));
}
GrTiledGradientEffect(const GrTiledGradientEffect& src);
std::unique_ptr<GrFragmentProcessor> clone() const override;
@@ -33,21 +35,24 @@
bool mirror;
bool makePremul;
bool colorsAreOpaque;
+ bool layoutPreservesOpacity;
private:
GrTiledGradientEffect(std::unique_ptr<GrFragmentProcessor> colorizer,
std::unique_ptr<GrFragmentProcessor> gradLayout,
bool mirror,
bool makePremul,
- bool colorsAreOpaque)
+ bool colorsAreOpaque,
+ bool layoutPreservesOpacity)
: INHERITED(kGrTiledGradientEffect_ClassID,
(OptimizationFlags)kCompatibleWithCoverageAsAlpha_OptimizationFlag |
- (colorsAreOpaque && gradLayout->preservesOpaqueInput()
+ (colorsAreOpaque && layoutPreservesOpacity
? kPreservesOpaqueInput_OptimizationFlag
: kNone_OptimizationFlags))
, mirror(mirror)
, makePremul(makePremul)
- , colorsAreOpaque(colorsAreOpaque) {
+ , colorsAreOpaque(colorsAreOpaque)
+ , layoutPreservesOpacity(layoutPreservesOpacity) {
this->registerChild(std::move(colorizer), SkSL::SampleUsage::Explicit());
this->registerChild(std::move(gradLayout), SkSL::SampleUsage::PassThrough());
}