Reland "Reland "Reland "Reland "Revert "Initial land of SkSL DSL."""""

This reverts commit 4129b6b.

Reason for revert: WASM breakage: https://task-driver.skia.org/td/UBRwnWYfbc5IwUWqtFMv

Original change's description:
> Revert "Reland "Reland "Reland "Revert "Initial land of SkSL DSL."""""
>
> This reverts commit 346dd53.
>
> Change-Id: I93bb18438cc6c2ad43d058d6c3f95bcc65d0cea9
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/343916
> Commit-Queue: Ethan Nicholas <[email protected]>
> Reviewed-by: John Stiles <[email protected]>

[email protected],[email protected],[email protected]

# Not skipping CQ checks because this is a reland.

Change-Id: If05145cf9d9c51f4c76fe523f6050a670b5da669
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/345169
Reviewed-by: Ethan Nicholas <[email protected]>
Commit-Queue: John Stiles <[email protected]>

Author: Ethan Nicholas

gn/sksl.gni

@@ -56,13 +56,6 @@ skia_sksl_sources = [
   "$_src/sksl/SkSLUtil.h",
   "$_src/sksl/SkSLVMGenerator.cpp",
   "$_src/sksl/SkSLVMGenerator.h",
-  "$_src/sksl/dsl/DSLBlock.cpp",
-  "$_src/sksl/dsl/DSLExpression.cpp",
-  "$_src/sksl/dsl/DSLStatement.cpp",
-  "$_src/sksl/dsl/DSLType.cpp",
-  "$_src/sksl/dsl/DSLVar.cpp",
-  "$_src/sksl/dsl/DSL_core.cpp",
-  "$_src/sksl/dsl/priv/DSLWriter.cpp",
   "$_src/sksl/ir/SkSLBinaryExpression.h",
   "$_src/sksl/ir/SkSLBlock.h",
   "$_src/sksl/ir/SkSLBoolLiteral.h",

gn/tests.gni

@@ -264,7 +264,6 @@ tests_sources = [
   "$_tests/SkResourceCacheTest.cpp",
   "$_tests/SkRuntimeEffectTest.cpp",
   "$_tests/SkSLCross.cpp",
-  "$_tests/SkSLDSLTest.cpp",
   "$_tests/SkSLFPTestbed.cpp",
   "$_tests/SkSLGLSLTestbed.cpp",
   "$_tests/SkSLInterpreterTest.cpp",

src/gpu/glsl/GrGLSLShaderBuilder.cpp

@@ -13,8 +13,6 @@
 #include "src/gpu/glsl/GrGLSLBlend.h"
 #include "src/gpu/glsl/GrGLSLColorSpaceXformHelper.h"
 #include "src/gpu/glsl/GrGLSLProgramBuilder.h"
-#include "src/sksl/dsl/DSL.h"
-#include "src/sksl/dsl/priv/DSLWriter.h"
 
 GrGLSLShaderBuilder::GrGLSLShaderBuilder(GrGLSLProgramBuilder* program)
     : fProgramBuilder(program)
@@ -83,13 +81,6 @@ void GrGLSLShaderBuilder::emitFunctionPrototype(GrSLType returnType,
     this->functions().append(";\n");
 }
 
-void GrGLSLShaderBuilder::codeAppend(SkSL::dsl::Statement stmt) {
-    std::unique_ptr<SkSL::Statement> skslStmt = stmt.release();
-    if (skslStmt) {
-        this->codeAppend(skslStmt->description().c_str());
-    }
-}
-
 static inline void append_texture_swizzle(SkString* out, GrSwizzle swizzle) {
     if (swizzle != GrSwizzle::RGBA()) {
         out->appendf(".%s", swizzle.asString().c_str());

src/gpu/glsl/GrGLSLShaderBuilder.h

@@ -17,14 +17,6 @@
 
 #include <stdarg.h>
 
-namespace SkSL {
-
-namespace dsl {
-    class DSLStatement;
-} // namespace dsl
-
-} // namespace SkSL
-
 class GrGLSLColorSpaceXformHelper;
 
 /**
@@ -117,8 +109,6 @@ class GrGLSLShaderBuilder {
 
     void codeAppend(const char* str, size_t length) { this->code().append(str, length); }
 
-    void codeAppend(SkSL::dsl::DSLStatement stmt);
-
     void codePrependf(const char format[], ...) SK_PRINTF_LIKE(2, 3) {
        va_list args;
        va_start(args, format);

src/sksl/SkSLCFGGenerator.cpp

@@ -366,9 +366,6 @@ void CFGGenerator::addExpression(CFG& cfg, std::unique_ptr<Expression>* e, bool
             }
             break;
         }
-        case Expression::Kind::kCodeString:
-            SkDEBUGFAIL("shouldn't be able to receive kCodeString here");
-            break;
         case Expression::Kind::kConstructor: {
             Constructor& c = e->get()->as<Constructor>();
             for (auto& arg : c.arguments()) {

src/sksl/SkSLCompiler.cpp

@@ -1813,7 +1813,6 @@ std::unique_ptr<Program> Compiler::convertProgram(
 
 bool Compiler::optimize(LoadedModule& module) {
     SkASSERT(!fErrorCount);
-    const Program::Settings* oldSettings = fIRGenerator->fSettings;
     Program::Settings settings;
     fIRGenerator->fKind = module.fKind;
     fIRGenerator->fSettings = &settings;
@@ -1838,7 +1837,6 @@ bool Compiler::optimize(LoadedModule& module) {
             break;
         }
     }
-    fIRGenerator->fSettings = oldSettings;
     return fErrorCount == 0;
 }
 
@@ -2161,13 +2159,10 @@ void Compiler::error(int offset, String msg) {
     fErrorText += "error: " + (pos.fLine >= 1 ? to_string(pos.fLine) + ": " : "") + msg + "\n";
 }
 
-String Compiler::errorText(bool showCount) {
-    if (showCount) {
-        this->writeErrorCount();
-    }
+String Compiler::errorText() {
+    this->writeErrorCount();
     fErrorCount = 0;
     String result = fErrorText;
-    fErrorText = "";
     return result;
 }
 

src/sksl/SkSLCompiler.h

@@ -45,11 +45,6 @@ class SkSLCompileBench;
 
 namespace SkSL {
 
-namespace dsl {
-    class DSL;
-    class DSLWriter;
-} // namespace dsl
-
 class ByteCode;
 class ExternalValue;
 class IRGenerator;
@@ -183,7 +178,7 @@ class SK_API Compiler : public ErrorReporter {
 
     void error(int offset, String msg) override;
 
-    String errorText(bool showCount = true);
+    String errorText();
 
     void writeErrorCount();
 
@@ -310,8 +305,6 @@ class SK_API Compiler : public ErrorReporter {
 
     friend class AutoSource;
     friend class ::SkSLCompileBench;
-    friend class dsl::DSL;
-    friend class dsl::DSLWriter;
 };
 
 #if !defined(SKSL_STANDALONE) && SK_SUPPORT_GPU

src/sksl/SkSLDehydrator.cpp

@@ -279,9 +279,6 @@ void Dehydrator::write(const Expression* e) {
                 this->writeU8(b.value());
                 break;
             }
-            case Expression::Kind::kCodeString:
-                SkDEBUGFAIL("shouldn't be able to receive kCodeString here");
-                break;
             case Expression::Kind::kConstructor: {
                 const Constructor& c = e->as<Constructor>();
                 this->writeCommand(Rehydrator::kConstructor_Command);