commit 4bdc12142a0e69fb7ae88076aed5fb48ab956507
author John Stiles <johnstiles@google.com> Mon Dec 14 18:04:13 2020 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Dec 15 14:03:59 2020 +0000
tree 94e22eb0a812bcf906bed5b546a9f205340cb2c5
parent 0765022c1517703c904e456352ef4e2d2973b32e

Use explicit return statements in more fragment processors.

Change-Id: I9c059ec562f981fdd5a2ac78d40aec204af2a5f9
Bug: skia:10549
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/344160
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: John Stiles <johnstiles@google.com>

gm/sample_matrix_variable.cpp

diff --git a/gm/sample_matrix_variable.cpp b/gm/sample_matrix_variable.cpp
index 3ca97b3..175e1d2 100644
--- a/gm/sample_matrix_variable.cpp
+++ b/gm/sample_matrix_variable.cpp
@@ -28,6 +28,7 @@
     }
 
     const char* name() const override { return "SampleMatrixVariableEffect"; }
+    bool usesExplicitReturn() const override { return true; }
 
     std::unique_ptr<GrFragmentProcessor> clone() const override {
         SkASSERT(false);
@@ -57,8 +58,7 @@
                                                                       "%g, %g, 1)",
                                                                       smve.fXOffset,
                                                                       smve.fYOffset).c_str());
-        fragBuilder->codeAppendf("%s = (%s + %s) / 2;\n", args.fOutputColor, sample1.c_str(),
-                                 sample2.c_str());
+        fragBuilder->codeAppendf("return (%s + %s) / 2;\n", sample1.c_str(), sample2.c_str());
     }
 };