1*c8dee2aaSAndroid Build Coastguard Worker /* 2*c8dee2aaSAndroid Build Coastguard Worker * Copyright 2021 Google LLC 3*c8dee2aaSAndroid Build Coastguard Worker * 4*c8dee2aaSAndroid Build Coastguard Worker * Use of this source code is governed by a BSD-style license that can be 5*c8dee2aaSAndroid Build Coastguard Worker * found in the LICENSE file. 6*c8dee2aaSAndroid Build Coastguard Worker */ 7*c8dee2aaSAndroid Build Coastguard Worker 8*c8dee2aaSAndroid Build Coastguard Worker #include "include/core/SkTypes.h" 9*c8dee2aaSAndroid Build Coastguard Worker #include "src/core/SkTHash.h" 10*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLAnalysis.h" 11*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLErrorReporter.h" 12*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLOperator.h" 13*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/analysis/SkSLProgramVisitor.h" 14*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLBinaryExpression.h" 15*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLExpression.h" 16*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLForStatement.h" 17*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLIRNode.h" 18*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLIndexExpression.h" 19*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLModifierFlags.h" 20*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLStatement.h" 21*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVarDeclarations.h" 22*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVariable.h" 23*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVariableReference.h" 24*c8dee2aaSAndroid Build Coastguard Worker 25*c8dee2aaSAndroid Build Coastguard Worker #include <memory> 26*c8dee2aaSAndroid Build Coastguard Worker 27*c8dee2aaSAndroid Build Coastguard Worker using namespace skia_private; 28*c8dee2aaSAndroid Build Coastguard Worker 29*c8dee2aaSAndroid Build Coastguard Worker namespace SkSL { 30*c8dee2aaSAndroid Build Coastguard Worker 31*c8dee2aaSAndroid Build Coastguard Worker class ProgramElement; 32*c8dee2aaSAndroid Build Coastguard Worker 33*c8dee2aaSAndroid Build Coastguard Worker namespace { 34*c8dee2aaSAndroid Build Coastguard Worker 35*c8dee2aaSAndroid Build Coastguard Worker // Checks for ES2 constant-expression rules, and (optionally) constant-index-expression rules 36*c8dee2aaSAndroid Build Coastguard Worker // (if loopIndices is non-nullptr) 37*c8dee2aaSAndroid Build Coastguard Worker class ConstantExpressionVisitor : public ProgramVisitor { 38*c8dee2aaSAndroid Build Coastguard Worker public: ConstantExpressionVisitor(const THashSet<const Variable * > * loopIndices)39*c8dee2aaSAndroid Build Coastguard Worker ConstantExpressionVisitor(const THashSet<const Variable*>* loopIndices) 40*c8dee2aaSAndroid Build Coastguard Worker : fLoopIndices(loopIndices) {} 41*c8dee2aaSAndroid Build Coastguard Worker visitExpression(const Expression & e)42*c8dee2aaSAndroid Build Coastguard Worker bool visitExpression(const Expression& e) override { 43*c8dee2aaSAndroid Build Coastguard Worker // A constant-(index)-expression is one of... 44*c8dee2aaSAndroid Build Coastguard Worker switch (e.kind()) { 45*c8dee2aaSAndroid Build Coastguard Worker // ... a literal value 46*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kLiteral: 47*c8dee2aaSAndroid Build Coastguard Worker return false; 48*c8dee2aaSAndroid Build Coastguard Worker 49*c8dee2aaSAndroid Build Coastguard Worker // ... settings can appear in fragment processors; they will resolve when compiled 50*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kSetting: 51*c8dee2aaSAndroid Build Coastguard Worker return false; 52*c8dee2aaSAndroid Build Coastguard Worker 53*c8dee2aaSAndroid Build Coastguard Worker // ... a global or local variable qualified as 'const', excluding function parameters. 54*c8dee2aaSAndroid Build Coastguard Worker // ... loop indices as defined in section 4. [constant-index-expression] 55*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kVariableReference: { 56*c8dee2aaSAndroid Build Coastguard Worker const Variable* v = e.as<VariableReference>().variable(); 57*c8dee2aaSAndroid Build Coastguard Worker if (v->modifierFlags().isConst() && (v->storage() == Variable::Storage::kGlobal || 58*c8dee2aaSAndroid Build Coastguard Worker v->storage() == Variable::Storage::kLocal)) { 59*c8dee2aaSAndroid Build Coastguard Worker return false; 60*c8dee2aaSAndroid Build Coastguard Worker } 61*c8dee2aaSAndroid Build Coastguard Worker return !fLoopIndices || !fLoopIndices->contains(v); 62*c8dee2aaSAndroid Build Coastguard Worker } 63*c8dee2aaSAndroid Build Coastguard Worker 64*c8dee2aaSAndroid Build Coastguard Worker // ... not a sequence expression (skia:13311)... 65*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kBinary: 66*c8dee2aaSAndroid Build Coastguard Worker if (e.as<BinaryExpression>().getOperator().kind() == Operator::Kind::COMMA) { 67*c8dee2aaSAndroid Build Coastguard Worker return true; 68*c8dee2aaSAndroid Build Coastguard Worker } 69*c8dee2aaSAndroid Build Coastguard Worker [[fallthrough]]; 70*c8dee2aaSAndroid Build Coastguard Worker 71*c8dee2aaSAndroid Build Coastguard Worker // ... expressions composed of both of the above 72*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorArray: 73*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorArrayCast: 74*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorCompound: 75*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorCompoundCast: 76*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorDiagonalMatrix: 77*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorMatrixResize: 78*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorScalarCast: 79*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorSplat: 80*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kConstructorStruct: 81*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kFieldAccess: 82*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kIndex: 83*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kPrefix: 84*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kPostfix: 85*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kSwizzle: 86*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kTernary: 87*c8dee2aaSAndroid Build Coastguard Worker return INHERITED::visitExpression(e); 88*c8dee2aaSAndroid Build Coastguard Worker 89*c8dee2aaSAndroid Build Coastguard Worker // Function calls are completely disallowed in SkSL constant-(index)-expressions. 90*c8dee2aaSAndroid Build Coastguard Worker // GLSL does mandate that calling a built-in function where the arguments are all 91*c8dee2aaSAndroid Build Coastguard Worker // constant-expressions should result in a constant-expression. SkSL handles this by 92*c8dee2aaSAndroid Build Coastguard Worker // optimizing fully-constant function calls into literals in FunctionCall::Make. 93*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kFunctionCall: 94*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kChildCall: 95*c8dee2aaSAndroid Build Coastguard Worker 96*c8dee2aaSAndroid Build Coastguard Worker // These shouldn't appear in a valid program at all, and definitely aren't 97*c8dee2aaSAndroid Build Coastguard Worker // constant-(index)-expressions. 98*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kPoison: 99*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kFunctionReference: 100*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kMethodReference: 101*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kTypeReference: 102*c8dee2aaSAndroid Build Coastguard Worker case Expression::Kind::kEmpty: 103*c8dee2aaSAndroid Build Coastguard Worker return true; 104*c8dee2aaSAndroid Build Coastguard Worker 105*c8dee2aaSAndroid Build Coastguard Worker default: 106*c8dee2aaSAndroid Build Coastguard Worker SkDEBUGFAIL("Unexpected expression type"); 107*c8dee2aaSAndroid Build Coastguard Worker return true; 108*c8dee2aaSAndroid Build Coastguard Worker } 109*c8dee2aaSAndroid Build Coastguard Worker } 110*c8dee2aaSAndroid Build Coastguard Worker 111*c8dee2aaSAndroid Build Coastguard Worker private: 112*c8dee2aaSAndroid Build Coastguard Worker const THashSet<const Variable*>* fLoopIndices; 113*c8dee2aaSAndroid Build Coastguard Worker using INHERITED = ProgramVisitor; 114*c8dee2aaSAndroid Build Coastguard Worker }; 115*c8dee2aaSAndroid Build Coastguard Worker 116*c8dee2aaSAndroid Build Coastguard Worker // Visits a function, tracks its loop indices, and verifies that every index-expression in the 117*c8dee2aaSAndroid Build Coastguard Worker // function qualifies as a constant-index-expression. 118*c8dee2aaSAndroid Build Coastguard Worker class ES2IndexingVisitor : public ProgramVisitor { 119*c8dee2aaSAndroid Build Coastguard Worker public: ES2IndexingVisitor(ErrorReporter & errors)120*c8dee2aaSAndroid Build Coastguard Worker ES2IndexingVisitor(ErrorReporter& errors) : fErrors(errors) {} 121*c8dee2aaSAndroid Build Coastguard Worker visitStatement(const Statement & s)122*c8dee2aaSAndroid Build Coastguard Worker bool visitStatement(const Statement& s) override { 123*c8dee2aaSAndroid Build Coastguard Worker if (s.is<ForStatement>()) { 124*c8dee2aaSAndroid Build Coastguard Worker const ForStatement& f = s.as<ForStatement>(); 125*c8dee2aaSAndroid Build Coastguard Worker SkASSERT(f.initializer() && f.initializer()->is<VarDeclaration>()); 126*c8dee2aaSAndroid Build Coastguard Worker const Variable* var = f.initializer()->as<VarDeclaration>().var(); 127*c8dee2aaSAndroid Build Coastguard Worker SkASSERT(!fLoopIndices.contains(var)); 128*c8dee2aaSAndroid Build Coastguard Worker fLoopIndices.add(var); 129*c8dee2aaSAndroid Build Coastguard Worker bool result = this->visitStatement(*f.statement()); 130*c8dee2aaSAndroid Build Coastguard Worker fLoopIndices.remove(var); 131*c8dee2aaSAndroid Build Coastguard Worker return result; 132*c8dee2aaSAndroid Build Coastguard Worker } 133*c8dee2aaSAndroid Build Coastguard Worker return INHERITED::visitStatement(s); 134*c8dee2aaSAndroid Build Coastguard Worker } 135*c8dee2aaSAndroid Build Coastguard Worker visitExpression(const Expression & e)136*c8dee2aaSAndroid Build Coastguard Worker bool visitExpression(const Expression& e) override { 137*c8dee2aaSAndroid Build Coastguard Worker if (e.is<IndexExpression>()) { 138*c8dee2aaSAndroid Build Coastguard Worker const IndexExpression& i = e.as<IndexExpression>(); 139*c8dee2aaSAndroid Build Coastguard Worker if (ConstantExpressionVisitor{&fLoopIndices}.visitExpression(*i.index())) { 140*c8dee2aaSAndroid Build Coastguard Worker fErrors.error(i.fPosition, "index expression must be constant"); 141*c8dee2aaSAndroid Build Coastguard Worker return true; 142*c8dee2aaSAndroid Build Coastguard Worker } 143*c8dee2aaSAndroid Build Coastguard Worker } 144*c8dee2aaSAndroid Build Coastguard Worker return INHERITED::visitExpression(e); 145*c8dee2aaSAndroid Build Coastguard Worker } 146*c8dee2aaSAndroid Build Coastguard Worker 147*c8dee2aaSAndroid Build Coastguard Worker using ProgramVisitor::visitProgramElement; 148*c8dee2aaSAndroid Build Coastguard Worker 149*c8dee2aaSAndroid Build Coastguard Worker private: 150*c8dee2aaSAndroid Build Coastguard Worker ErrorReporter& fErrors; 151*c8dee2aaSAndroid Build Coastguard Worker THashSet<const Variable*> fLoopIndices; 152*c8dee2aaSAndroid Build Coastguard Worker using INHERITED = ProgramVisitor; 153*c8dee2aaSAndroid Build Coastguard Worker }; 154*c8dee2aaSAndroid Build Coastguard Worker 155*c8dee2aaSAndroid Build Coastguard Worker } // namespace 156*c8dee2aaSAndroid Build Coastguard Worker IsConstantExpression(const Expression & expr)157*c8dee2aaSAndroid Build Coastguard Workerbool Analysis::IsConstantExpression(const Expression& expr) { 158*c8dee2aaSAndroid Build Coastguard Worker return !ConstantExpressionVisitor{/*loopIndices=*/nullptr}.visitExpression(expr); 159*c8dee2aaSAndroid Build Coastguard Worker } 160*c8dee2aaSAndroid Build Coastguard Worker ValidateIndexingForES2(const ProgramElement & pe,ErrorReporter & errors)161*c8dee2aaSAndroid Build Coastguard Workervoid Analysis::ValidateIndexingForES2(const ProgramElement& pe, ErrorReporter& errors) { 162*c8dee2aaSAndroid Build Coastguard Worker ES2IndexingVisitor visitor(errors); 163*c8dee2aaSAndroid Build Coastguard Worker visitor.visitProgramElement(pe); 164*c8dee2aaSAndroid Build Coastguard Worker } 165*c8dee2aaSAndroid Build Coastguard Worker 166*c8dee2aaSAndroid Build Coastguard Worker } // namespace SkSL 167