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/SkSpan.h"
9*c8dee2aaSAndroid Build Coastguard Worker #include "include/core/SkTypes.h"
10*c8dee2aaSAndroid Build Coastguard Worker #include "src/core/SkTHash.h"
11*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLAnalysis.h"
12*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLModule.h"
13*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/SkSLProgramSettings.h"
14*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/analysis/SkSLProgramUsage.h"
15*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLBinaryExpression.h"
16*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLExpression.h"
17*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLExpressionStatement.h"
18*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLFunctionDefinition.h"
19*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLNop.h"
20*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLProgram.h"
21*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLProgramElement.h"
22*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLStatement.h"
23*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVarDeclarations.h"
24*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVariable.h"
25*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/ir/SkSLVariableReference.h"
26*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/transform/SkSLProgramWriter.h"
27*c8dee2aaSAndroid Build Coastguard Worker #include "src/sksl/transform/SkSLTransform.h"
28*c8dee2aaSAndroid Build Coastguard Worker
29*c8dee2aaSAndroid Build Coastguard Worker #include <memory>
30*c8dee2aaSAndroid Build Coastguard Worker #include <utility>
31*c8dee2aaSAndroid Build Coastguard Worker #include <vector>
32*c8dee2aaSAndroid Build Coastguard Worker
33*c8dee2aaSAndroid Build Coastguard Worker using namespace skia_private;
34*c8dee2aaSAndroid Build Coastguard Worker
35*c8dee2aaSAndroid Build Coastguard Worker namespace SkSL {
36*c8dee2aaSAndroid Build Coastguard Worker
37*c8dee2aaSAndroid Build Coastguard Worker class Context;
38*c8dee2aaSAndroid Build Coastguard Worker
eliminate_dead_local_variables(const Context & context,SkSpan<std::unique_ptr<ProgramElement>> elements,ProgramUsage * usage)39*c8dee2aaSAndroid Build Coastguard Worker static bool eliminate_dead_local_variables(const Context& context,
40*c8dee2aaSAndroid Build Coastguard Worker SkSpan<std::unique_ptr<ProgramElement>> elements,
41*c8dee2aaSAndroid Build Coastguard Worker ProgramUsage* usage) {
42*c8dee2aaSAndroid Build Coastguard Worker class DeadLocalVariableEliminator : public ProgramWriter {
43*c8dee2aaSAndroid Build Coastguard Worker public:
44*c8dee2aaSAndroid Build Coastguard Worker DeadLocalVariableEliminator(const Context& context, ProgramUsage* usage)
45*c8dee2aaSAndroid Build Coastguard Worker : fContext(context)
46*c8dee2aaSAndroid Build Coastguard Worker , fUsage(usage) {}
47*c8dee2aaSAndroid Build Coastguard Worker
48*c8dee2aaSAndroid Build Coastguard Worker using ProgramWriter::visitProgramElement;
49*c8dee2aaSAndroid Build Coastguard Worker
50*c8dee2aaSAndroid Build Coastguard Worker bool visitExpressionPtr(std::unique_ptr<Expression>& expr) override {
51*c8dee2aaSAndroid Build Coastguard Worker if (expr->is<BinaryExpression>()) {
52*c8dee2aaSAndroid Build Coastguard Worker // Search for expressions of the form `deadVar = anyExpression`.
53*c8dee2aaSAndroid Build Coastguard Worker BinaryExpression& binary = expr->as<BinaryExpression>();
54*c8dee2aaSAndroid Build Coastguard Worker if (VariableReference* assignedVar = binary.isAssignmentIntoVariable()) {
55*c8dee2aaSAndroid Build Coastguard Worker if (fDeadVariables.contains(assignedVar->variable())) {
56*c8dee2aaSAndroid Build Coastguard Worker // Replace `deadVar = anyExpression` with `anyExpression`.
57*c8dee2aaSAndroid Build Coastguard Worker fUsage->remove(expr.get());
58*c8dee2aaSAndroid Build Coastguard Worker expr = std::move(binary.right());
59*c8dee2aaSAndroid Build Coastguard Worker fUsage->add(expr.get());
60*c8dee2aaSAndroid Build Coastguard Worker
61*c8dee2aaSAndroid Build Coastguard Worker // If `anyExpression` is now a lone ExpressionStatement, it's highly likely
62*c8dee2aaSAndroid Build Coastguard Worker // that we can eliminate it entirely. This flag will let us know to check.
63*c8dee2aaSAndroid Build Coastguard Worker fAssignmentWasEliminated = true;
64*c8dee2aaSAndroid Build Coastguard Worker
65*c8dee2aaSAndroid Build Coastguard Worker // Re-process the newly cleaned-up expression. This lets us fully clean up
66*c8dee2aaSAndroid Build Coastguard Worker // gnarly assignments like `a = b = 123;` where both `a` and `b` are dead,
67*c8dee2aaSAndroid Build Coastguard Worker // or silly double-assignments like `a = a = 123;`.
68*c8dee2aaSAndroid Build Coastguard Worker return this->visitExpressionPtr(expr);
69*c8dee2aaSAndroid Build Coastguard Worker }
70*c8dee2aaSAndroid Build Coastguard Worker }
71*c8dee2aaSAndroid Build Coastguard Worker }
72*c8dee2aaSAndroid Build Coastguard Worker if (expr->is<VariableReference>()) {
73*c8dee2aaSAndroid Build Coastguard Worker SkASSERT(!fDeadVariables.contains(expr->as<VariableReference>().variable()));
74*c8dee2aaSAndroid Build Coastguard Worker }
75*c8dee2aaSAndroid Build Coastguard Worker return INHERITED::visitExpressionPtr(expr);
76*c8dee2aaSAndroid Build Coastguard Worker }
77*c8dee2aaSAndroid Build Coastguard Worker
78*c8dee2aaSAndroid Build Coastguard Worker bool visitStatementPtr(std::unique_ptr<Statement>& stmt) override {
79*c8dee2aaSAndroid Build Coastguard Worker if (stmt->is<VarDeclaration>()) {
80*c8dee2aaSAndroid Build Coastguard Worker VarDeclaration& varDecl = stmt->as<VarDeclaration>();
81*c8dee2aaSAndroid Build Coastguard Worker const Variable* var = varDecl.var();
82*c8dee2aaSAndroid Build Coastguard Worker ProgramUsage::VariableCounts* counts = fUsage->fVariableCounts.find(var);
83*c8dee2aaSAndroid Build Coastguard Worker SkASSERT(counts);
84*c8dee2aaSAndroid Build Coastguard Worker SkASSERT(counts->fVarExists);
85*c8dee2aaSAndroid Build Coastguard Worker if (CanEliminate(var, *counts)) {
86*c8dee2aaSAndroid Build Coastguard Worker fDeadVariables.add(var);
87*c8dee2aaSAndroid Build Coastguard Worker if (var->initialValue()) {
88*c8dee2aaSAndroid Build Coastguard Worker // The variable has an initial-value expression, which might have side
89*c8dee2aaSAndroid Build Coastguard Worker // effects. ExpressionStatement::Make will preserve side effects, but
90*c8dee2aaSAndroid Build Coastguard Worker // replaces pure expressions with Nop.
91*c8dee2aaSAndroid Build Coastguard Worker fUsage->remove(stmt.get());
92*c8dee2aaSAndroid Build Coastguard Worker stmt = ExpressionStatement::Make(fContext, std::move(varDecl.value()));
93*c8dee2aaSAndroid Build Coastguard Worker fUsage->add(stmt.get());
94*c8dee2aaSAndroid Build Coastguard Worker } else {
95*c8dee2aaSAndroid Build Coastguard Worker // The variable has no initial-value and can be cleanly eliminated.
96*c8dee2aaSAndroid Build Coastguard Worker fUsage->remove(stmt.get());
97*c8dee2aaSAndroid Build Coastguard Worker stmt = Nop::Make();
98*c8dee2aaSAndroid Build Coastguard Worker }
99*c8dee2aaSAndroid Build Coastguard Worker fMadeChanges = true;
100*c8dee2aaSAndroid Build Coastguard Worker
101*c8dee2aaSAndroid Build Coastguard Worker // Re-process the newly cleaned-up statement. This lets us fully clean up
102*c8dee2aaSAndroid Build Coastguard Worker // gnarly assignments like `a = b = 123;` where both `a` and `b` are dead,
103*c8dee2aaSAndroid Build Coastguard Worker // or silly double-assignments like `a = a = 123;`.
104*c8dee2aaSAndroid Build Coastguard Worker return this->visitStatementPtr(stmt);
105*c8dee2aaSAndroid Build Coastguard Worker }
106*c8dee2aaSAndroid Build Coastguard Worker }
107*c8dee2aaSAndroid Build Coastguard Worker
108*c8dee2aaSAndroid Build Coastguard Worker bool result = INHERITED::visitStatementPtr(stmt);
109*c8dee2aaSAndroid Build Coastguard Worker
110*c8dee2aaSAndroid Build Coastguard Worker // If we eliminated an assignment above, we may have left behind an inert
111*c8dee2aaSAndroid Build Coastguard Worker // ExpressionStatement.
112*c8dee2aaSAndroid Build Coastguard Worker if (fAssignmentWasEliminated) {
113*c8dee2aaSAndroid Build Coastguard Worker fAssignmentWasEliminated = false;
114*c8dee2aaSAndroid Build Coastguard Worker if (stmt->is<ExpressionStatement>()) {
115*c8dee2aaSAndroid Build Coastguard Worker ExpressionStatement& exprStmt = stmt->as<ExpressionStatement>();
116*c8dee2aaSAndroid Build Coastguard Worker if (!Analysis::HasSideEffects(*exprStmt.expression())) {
117*c8dee2aaSAndroid Build Coastguard Worker // The expression-statement was inert; eliminate it entirely.
118*c8dee2aaSAndroid Build Coastguard Worker fUsage->remove(&exprStmt);
119*c8dee2aaSAndroid Build Coastguard Worker stmt = Nop::Make();
120*c8dee2aaSAndroid Build Coastguard Worker }
121*c8dee2aaSAndroid Build Coastguard Worker }
122*c8dee2aaSAndroid Build Coastguard Worker }
123*c8dee2aaSAndroid Build Coastguard Worker
124*c8dee2aaSAndroid Build Coastguard Worker return result;
125*c8dee2aaSAndroid Build Coastguard Worker }
126*c8dee2aaSAndroid Build Coastguard Worker
127*c8dee2aaSAndroid Build Coastguard Worker static bool CanEliminate(const Variable* var, const ProgramUsage::VariableCounts& counts) {
128*c8dee2aaSAndroid Build Coastguard Worker return counts.fVarExists && !counts.fRead && var->storage() == VariableStorage::kLocal;
129*c8dee2aaSAndroid Build Coastguard Worker }
130*c8dee2aaSAndroid Build Coastguard Worker
131*c8dee2aaSAndroid Build Coastguard Worker bool fMadeChanges = false;
132*c8dee2aaSAndroid Build Coastguard Worker const Context& fContext;
133*c8dee2aaSAndroid Build Coastguard Worker ProgramUsage* fUsage;
134*c8dee2aaSAndroid Build Coastguard Worker THashSet<const Variable*> fDeadVariables;
135*c8dee2aaSAndroid Build Coastguard Worker bool fAssignmentWasEliminated = false;
136*c8dee2aaSAndroid Build Coastguard Worker
137*c8dee2aaSAndroid Build Coastguard Worker using INHERITED = ProgramWriter;
138*c8dee2aaSAndroid Build Coastguard Worker };
139*c8dee2aaSAndroid Build Coastguard Worker
140*c8dee2aaSAndroid Build Coastguard Worker DeadLocalVariableEliminator visitor{context, usage};
141*c8dee2aaSAndroid Build Coastguard Worker
142*c8dee2aaSAndroid Build Coastguard Worker for (auto& [var, counts] : usage->fVariableCounts) {
143*c8dee2aaSAndroid Build Coastguard Worker if (DeadLocalVariableEliminator::CanEliminate(var, counts)) {
144*c8dee2aaSAndroid Build Coastguard Worker // This program contains at least one dead local variable.
145*c8dee2aaSAndroid Build Coastguard Worker // Scan the program for any dead local variables and eliminate them all.
146*c8dee2aaSAndroid Build Coastguard Worker for (std::unique_ptr<ProgramElement>& pe : elements) {
147*c8dee2aaSAndroid Build Coastguard Worker if (pe->is<FunctionDefinition>()) {
148*c8dee2aaSAndroid Build Coastguard Worker visitor.visitProgramElement(*pe);
149*c8dee2aaSAndroid Build Coastguard Worker }
150*c8dee2aaSAndroid Build Coastguard Worker }
151*c8dee2aaSAndroid Build Coastguard Worker break;
152*c8dee2aaSAndroid Build Coastguard Worker }
153*c8dee2aaSAndroid Build Coastguard Worker }
154*c8dee2aaSAndroid Build Coastguard Worker
155*c8dee2aaSAndroid Build Coastguard Worker return visitor.fMadeChanges;
156*c8dee2aaSAndroid Build Coastguard Worker }
157*c8dee2aaSAndroid Build Coastguard Worker
EliminateDeadLocalVariables(const Context & context,Module & module,ProgramUsage * usage)158*c8dee2aaSAndroid Build Coastguard Worker bool Transform::EliminateDeadLocalVariables(const Context& context,
159*c8dee2aaSAndroid Build Coastguard Worker Module& module,
160*c8dee2aaSAndroid Build Coastguard Worker ProgramUsage* usage) {
161*c8dee2aaSAndroid Build Coastguard Worker return eliminate_dead_local_variables(context, SkSpan(module.fElements), usage);
162*c8dee2aaSAndroid Build Coastguard Worker }
163*c8dee2aaSAndroid Build Coastguard Worker
EliminateDeadLocalVariables(Program & program)164*c8dee2aaSAndroid Build Coastguard Worker bool Transform::EliminateDeadLocalVariables(Program& program) {
165*c8dee2aaSAndroid Build Coastguard Worker return program.fConfig->fSettings.fRemoveDeadVariables
166*c8dee2aaSAndroid Build Coastguard Worker ? eliminate_dead_local_variables(*program.fContext,
167*c8dee2aaSAndroid Build Coastguard Worker SkSpan(program.fOwnedElements),
168*c8dee2aaSAndroid Build Coastguard Worker program.fUsage.get())
169*c8dee2aaSAndroid Build Coastguard Worker : false;
170*c8dee2aaSAndroid Build Coastguard Worker }
171*c8dee2aaSAndroid Build Coastguard Worker
172*c8dee2aaSAndroid Build Coastguard Worker } // namespace SkSL
173