Wrap SkShader::program calls with #ifdef SK_ENABLE_SKVM.

Change-Id: I3ad69746ecd66c60d2b77fc6e2314727ade227bc Reviewed-on: https://skia-review.googlesource.com/c/skia/+/674656 Commit-Queue: Herb Derby <[email protected]> Reviewed-by: Herb Derby <[email protected]> Auto-Submit: John Stiles <[email protected]>
armansito · Apr 20, 2023 · 3acd1bf · 3acd1bf
1 parent 024c075
commit 3acd1bf
Show file tree

Hide file tree

Showing 22 changed files with 80 additions and 18 deletions.
diff --git a/src/core/SkDraw_atlas.cpp b/src/core/SkDraw_atlas.cpp
@@ -35,7 +35,6 @@
 #include "src/core/SkRasterPipelineOpList.h"
 #include "src/core/SkScan.h"
 #include "src/core/SkSurfacePriv.h"
-#include "src/core/SkVM.h"
 #include "src/core/SkVMBlitter.h"
 #include "src/shaders/SkShaderBase.h"
 #include "src/shaders/SkTransformShader.h"
@@ -45,10 +44,13 @@
 #include <utility>
 
 class SkBlitter;
-class SkColorInfo;
 class SkColorSpace;
 enum class SkBlendMode;
 
+#if defined(SK_ENABLE_SKVM)
+#include "src/core/SkVM.h"
+class SkColorInfo;
+#endif
 
 static void fill_rect(const SkMatrix& ctm, const SkRasterClip& rc,
                       const SkRect& r, SkBlitter* blitter, SkPath* scratchPath) {
@@ -81,6 +83,7 @@ class UpdatableColorShader : public SkShaderBase {
 public:
     explicit UpdatableColorShader(SkColorSpace* cs)
         : fSteps{sk_srgb_singleton(), kUnpremul_SkAlphaType, cs, kUnpremul_SkAlphaType} {}
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder* builder,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -97,6 +100,7 @@ class UpdatableColorShader : public SkShaderBase {
 
         return {r, g, b, a};
     }
+#endif
 
     void updateColor(SkColor c) const {
         SkColor4f c4 = SkColor4f::FromColor(c);

diff --git a/src/core/SkDraw_vertices.cpp b/src/core/SkDraw_vertices.cpp
@@ -41,7 +41,6 @@
 #include "src/core/SkRasterPipelineOpList.h"
 #include "src/core/SkScan.h"
 #include "src/core/SkSurfacePriv.h"
-#include "src/core/SkVM.h"
 #include "src/core/SkVMBlitter.h"
 #include "src/core/SkVertState.h"
 #include "src/core/SkVerticesPriv.h"
@@ -55,6 +54,10 @@
 
 class SkBlitter;
 
+#if defined(SK_ENABLE_SKVM)
+#include "src/core/SkVM.h"
+#endif
+
 struct Matrix43 {
     float fMat[12];    // column major
 
@@ -122,6 +125,7 @@ class SkTriColorShader : public SkShaderBase {
         return true;
     }
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord,
                         skvm::Coord,
@@ -130,6 +134,7 @@ class SkTriColorShader : public SkShaderBase {
                         const SkColorInfo&,
                         skvm::Uniforms*,
                         SkArenaAlloc*) const override;
+#endif
 
 private:
     bool isOpaque() const override { return fIsOpaque; }
@@ -144,12 +149,15 @@ class SkTriColorShader : public SkShaderBase {
     SkMatrix fM33;
     const bool fIsOpaque;
     const bool fUsePersp;   // controls our stages, and what we do in update()
+#if defined(SK_ENABLE_SKVM)
     mutable skvm::Uniform fColorMatrix;
     mutable skvm::Uniform fCoordMatrix;
+#endif
 
     using INHERITED = SkShaderBase;
 };
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkTriColorShader::program(skvm::Builder* b,
                                       skvm::Coord device,
                                       skvm::Coord local,
@@ -190,6 +198,7 @@ skvm::Color SkTriColorShader::program(skvm::Builder* b,
     color.a = colorDot(3);
     return color;
 }
+#endif
 
 bool SkTriColorShader::update(const SkMatrix& ctmInv, const SkPoint pts[],
                               const SkPMColor4f colors[], int index0, int index1, int index2) {

diff --git a/src/core/SkRuntimeEffect.cpp b/src/core/SkRuntimeEffect.cpp
@@ -438,7 +438,7 @@ static void write_child_effects(SkWriteBuffer& buffer,
     }
 }
 
-#ifndef SK_ENABLE_SKSL_IN_RASTER_PIPELINE
+#ifdef SK_ENABLE_SKVM
 static std::vector<skvm::Val> make_skvm_uniforms(skvm::Builder* p,
                                                  skvm::Uniforms* uniforms,
                                                  size_t inputSize,
@@ -952,6 +952,7 @@ class RuntimeEffectVMCallbacks : public SkSL::SkVMCallbacks {
             , fColorInfo(colorInfo) {}
 
     skvm::Color sampleShader(int ix, skvm::Coord coord) override {
+#if defined(SK_ENABLE_SKVM)
         // We haven't tracked device coords and the runtime effect could have arbitrarily
         // manipulated the passed coords. We should be in a state where any pending matrix was
         // already applied before the runtime effect's code could have manipulated the coords
@@ -968,6 +969,7 @@ class RuntimeEffectVMCallbacks : public SkSL::SkVMCallbacks {
                                           fUniforms,
                                           fAlloc);
         }
+#endif
         return fInColor;
     }
 
@@ -1091,9 +1093,7 @@ class SkRuntimeColorFilter : public SkColorFilterBase {
     skvm::Color onProgram(skvm::Builder* p, skvm::Color c,
                           const SkColorInfo& colorInfo,
                           skvm::Uniforms* uniforms, SkArenaAlloc* alloc) const override {
-#ifdef SK_ENABLE_SKSL_IN_RASTER_PIPELINE
-        return {};
-#else
+#ifdef SK_ENABLE_SKVM
         SkASSERT(SkRuntimeEffectPriv::CanDraw(SkCapabilities::RasterBackend().get(),
                                               fEffect.get()));
 
@@ -1115,6 +1115,8 @@ class SkRuntimeColorFilter : public SkColorFilterBase {
         return SkSL::ProgramToSkVM(*fEffect->fBaseProgram, fEffect->fMain, p,/*debugTrace=*/nullptr,
                                    SkSpan(uniform), /*device=*/zeroCoord, /*local=*/zeroCoord,
                                    c, c, &callbacks);
+#else
+        return {};
 #endif
     }
 
@@ -1315,6 +1317,7 @@ class SkRTShader : public SkShaderBase {
         return false;
     }
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder* p,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -1323,9 +1326,6 @@ class SkRTShader : public SkShaderBase {
                         const SkColorInfo& colorInfo,
                         skvm::Uniforms* uniforms,
                         SkArenaAlloc* alloc) const override {
-#ifdef SK_ENABLE_SKSL_IN_RASTER_PIPELINE
-        return {};
-#else
         if (!SkRuntimeEffectPriv::CanDraw(SkCapabilities::RasterBackend().get(), fEffect.get())) {
             return {};
         }
@@ -1357,8 +1357,8 @@ class SkRTShader : public SkShaderBase {
 
         return SkSL::ProgramToSkVM(*fEffect->fBaseProgram, fEffect->fMain, p, fDebugTrace.get(),
                                    SkSpan(uniform), device, local, paint, paint, &callbacks);
-#endif
     }
+#endif
 
     void flatten(SkWriteBuffer& buffer) const override {
         buffer.writeString(fEffect->source().c_str());
@@ -1475,9 +1475,7 @@ class SkRuntimeBlender : public SkBlenderBase {
     skvm::Color onProgram(skvm::Builder* p, skvm::Color src, skvm::Color dst,
                           const SkColorInfo& colorInfo, skvm::Uniforms* uniforms,
                           SkArenaAlloc* alloc) const override {
-#ifdef SK_ENABLE_SKSL_IN_RASTER_PIPELINE
-        return {};
-#else
+#ifdef SK_ENABLE_SKVM
         if (!SkRuntimeEffectPriv::CanDraw(SkCapabilities::RasterBackend().get(), fEffect.get())) {
             return {};
         }
@@ -1498,6 +1496,8 @@ class SkRuntimeBlender : public SkBlenderBase {
         return SkSL::ProgramToSkVM(*fEffect->fBaseProgram, fEffect->fMain, p,/*debugTrace=*/nullptr,
                                    SkSpan(uniform), /*device=*/zeroCoord, /*local=*/zeroCoord,
                                    src, dst, &callbacks);
+#else
+        return {};
 #endif
     }
 

diff --git a/src/shaders/SkColorFilterShader.cpp b/src/shaders/SkColorFilterShader.cpp
@@ -68,6 +68,7 @@ bool SkColorFilterShader::appendStages(const SkStageRec& rec, const MatrixRec& m
     return true;
 }
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkColorFilterShader::program(skvm::Builder* p,
                                          skvm::Coord device,
                                          skvm::Coord local,
@@ -93,7 +94,7 @@ skvm::Color SkColorFilterShader::program(skvm::Builder* p,
     // Finally run that through the color filter.
     return fFilter->program(p,c, dst, uniforms,alloc);
 }
-
+#endif
 #if defined(SK_GANESH)
 /////////////////////////////////////////////////////////////////////
 

diff --git a/src/shaders/SkColorFilterShader.h b/src/shaders/SkColorFilterShader.h
@@ -32,6 +32,7 @@ class SkColorFilterShader : public SkShaderBase {
     void flatten(SkWriteBuffer&) const override;
     bool appendStages(const SkStageRec&, const MatrixRec&) const override;
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -40,6 +41,7 @@ class SkColorFilterShader : public SkShaderBase {
                         const SkColorInfo& dst,
                         skvm::Uniforms* uniforms,
                         SkArenaAlloc*) const override;
+#endif
 
     SK_FLATTENABLE_HOOKS(SkColorFilterShader)
 

diff --git a/src/shaders/SkColorShader.cpp b/src/shaders/SkColorShader.cpp
@@ -64,6 +64,7 @@ class SkColorShader : public SkShaderBase {
 
     bool appendStages(const SkStageRec&, const MatrixRec&) const override;
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -72,6 +73,7 @@ class SkColorShader : public SkShaderBase {
                         const SkColorInfo& dst,
                         skvm::Uniforms* uniforms,
                         SkArenaAlloc*) const override;
+#endif
 
     SkColor fColor;
 };
@@ -100,6 +102,7 @@ class SkColor4Shader : public SkShaderBase {
     void flatten(SkWriteBuffer&) const override;
     bool appendStages(const SkStageRec&, const MatrixRec&) const override;
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -108,6 +111,7 @@ class SkColor4Shader : public SkShaderBase {
                         const SkColorInfo& dst,
                         skvm::Uniforms* uniforms,
                         SkArenaAlloc*) const override;
+#endif
 
     sk_sp<SkColorSpace> fColorSpace;
     const SkColor4f     fColor;
@@ -185,6 +189,7 @@ bool SkColor4Shader::appendStages(const SkStageRec& rec, const MatrixRec&) const
     return true;
 }
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkColorShader::program(skvm::Builder* p,
                                    skvm::Coord /*device*/,
                                    skvm::Coord /*local*/,
@@ -198,6 +203,7 @@ skvm::Color SkColorShader::program(skvm::Builder* p,
                               dst.colorSpace(),   kPremul_SkAlphaType).apply(color.vec());
     return p->uniformColor(color, uniforms);
 }
+
 skvm::Color SkColor4Shader::program(skvm::Builder* p,
                                     skvm::Coord /*device*/,
                                     skvm::Coord /*local*/,
@@ -211,6 +217,7 @@ skvm::Color SkColor4Shader::program(skvm::Builder* p,
                             dst.colorSpace(),   kPremul_SkAlphaType).apply(color.vec());
     return p->uniformColor(color, uniforms);
 }
+#endif  // defined(SK_ENABLE_SKVM)
 
 #if defined(SK_GANESH)
 

diff --git a/src/shaders/SkComposeShader.cpp b/src/shaders/SkComposeShader.cpp
@@ -48,6 +48,8 @@ class SkShader_Blend final : public SkShaderBase {
     SkShader_Blend(SkReadBuffer&);
     void flatten(SkWriteBuffer&) const override;
     bool appendStages(const SkStageRec&, const MatrixRec&) const override;
+
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -56,6 +58,7 @@ class SkShader_Blend final : public SkShaderBase {
                         const SkColorInfo& dst,
                         skvm::Uniforms*,
                         SkArenaAlloc*) const override;
+#endif  // defined(SK_ENABLE_SKVM)
 
 private:
     friend void ::SkRegisterComposeShaderFlattenable();
@@ -139,6 +142,7 @@ bool SkShader_Blend::appendStages(const SkStageRec& rec, const MatrixRec& mRec)
     return true;
 }
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkShader_Blend::program(skvm::Builder* p,
                                     skvm::Coord device,
                                     skvm::Coord local,
@@ -154,6 +158,7 @@ skvm::Color SkShader_Blend::program(skvm::Builder* p,
     }
     return {};
 }
+#endif
 
 #if defined(SK_GANESH)
 

diff --git a/src/shaders/SkCoordClampShader.cpp b/src/shaders/SkCoordClampShader.cpp
@@ -46,6 +46,7 @@ class SkShader_CoordClamp final : public SkShaderBase {
     SkShader_CoordClamp(SkReadBuffer&);
     void flatten(SkWriteBuffer&) const override;
     bool appendStages(const SkStageRec&, const MatrixRec&) const override;
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord device,
                         skvm::Coord local,
@@ -54,6 +55,7 @@ class SkShader_CoordClamp final : public SkShaderBase {
                         const SkColorInfo& dst,
                         skvm::Uniforms*,
                         SkArenaAlloc*) const override;
+#endif
 
 private:
     friend void ::SkRegisterCoordClampShaderFlattenable();
@@ -91,6 +93,7 @@ bool SkShader_CoordClamp::appendStages(const SkStageRec& rec, const MatrixRec& m
     return as_SB(fShader)->appendStages(rec, *childMRec);
 }
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkShader_CoordClamp::program(skvm::Builder* p,
                                          skvm::Coord device,
                                          skvm::Coord local,
@@ -115,6 +118,7 @@ skvm::Color SkShader_CoordClamp::program(skvm::Builder* p,
 
     return as_SB(fShader)->program(p, device, local, paint, *childMRec, cinfo, uniforms, alloc);
 }
+#endif
 
 #if defined(SK_GANESH)
 std::unique_ptr<GrFragmentProcessor> SkShader_CoordClamp::asFragmentProcessor(

diff --git a/src/shaders/SkEmptyShader.cpp b/src/shaders/SkEmptyShader.cpp
@@ -27,6 +27,7 @@ class SkEmptyShader : public SkShaderBase {
 
     bool appendStages(const SkStageRec&, const MatrixRec&) const override { return false; }
 
+#if defined(SK_ENABLE_SKVM)
     skvm::Color program(skvm::Builder*,
                         skvm::Coord,
                         skvm::Coord,
@@ -35,6 +36,7 @@ class SkEmptyShader : public SkShaderBase {
                         const SkColorInfo&,
                         skvm::Uniforms*,
                         SkArenaAlloc*) const override;
+#endif
 
 private:
     friend void ::SkRegisterEmptyShaderFlattenable();
@@ -43,6 +45,7 @@ class SkEmptyShader : public SkShaderBase {
     using INHERITED = SkShaderBase;
 };
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkEmptyShader::program(skvm::Builder*,
                                    skvm::Coord,
                                    skvm::Coord,
@@ -53,6 +56,7 @@ skvm::Color SkEmptyShader::program(skvm::Builder*,
                                    SkArenaAlloc*) const {
     return {};  // signal failure
 }
+#endif
 
 sk_sp<SkFlattenable> SkEmptyShader::CreateProc(SkReadBuffer&) {
     return SkShaders::Empty();

diff --git a/src/shaders/SkImageShader.cpp b/src/shaders/SkImageShader.cpp
@@ -854,6 +854,7 @@ bool SkImageShader::appendStages(const SkStageRec& rec, const MatrixRec& mRec) c
     return append_misc();
 }
 
+#if defined(SK_ENABLE_SKVM)
 skvm::Color SkImageShader::program(skvm::Builder* p,
                                    skvm::Coord device,
                                    skvm::Coord origLocal,
@@ -1142,3 +1143,4 @@ skvm::Color SkImageShader::program(skvm::Builder* p,
                 : SkColorSpaceXformSteps{cs, at, dst.colorSpace(), dst.alphaType()}.program(
                           p, uniforms, c);
 }
+#endif