Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

Unified Diff: src/sksl/SkSLCFGGenerator.h

Issue 2405383003: added basic dataflow analysis to skslc (Closed)
Patch Set: I have no idea what I was thinking Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/sksl/README ('k') | src/sksl/SkSLCFGGenerator.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/sksl/SkSLCFGGenerator.h
diff --git a/src/sksl/SkSLCFGGenerator.h b/src/sksl/SkSLCFGGenerator.h
new file mode 100644
index 0000000000000000000000000000000000000000..c37850112c0dbe0084ab4a08b64fa1c63c916485
--- /dev/null
+++ b/src/sksl/SkSLCFGGenerator.h
@@ -0,0 +1,90 @@
+/*
+ * Copyright 2016 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#ifndef SKSL_CFGGENERATOR
+#define SKSL_CFGGENERATOR
+
+#include "ir/SkSLExpression.h"
+#include "ir/SkSLFunctionDefinition.h"
+
+#include <set>
+#include <stack>
+
+namespace SkSL {
+
+// index of a block within CFG.fBlocks
+typedef size_t BlockId;
+
+struct BasicBlock {
+ struct Node {
+ enum Kind {
+ kStatement_Kind,
+ kExpression_Kind
+ };
+
+ Kind fKind;
+ const IRNode* fNode;
+ };
+
+ std::vector<Node> fNodes;
+ std::set<BlockId> fEntrances;
+ std::set<BlockId> fExits;
+ // variable definitions upon entering this basic block (null expression = undefined)
+ std::unordered_map<const Variable*, const Expression*> fBefore;
+};
+
+struct CFG {
+ BlockId fStart;
+ BlockId fExit;
+ std::vector<BasicBlock> fBlocks;
+
+ void dump();
+
+private:
+ BlockId fCurrent;
+
+ // Adds a new block, adds an exit* from the current block to the new block, then marks the new
+ // block as the current block
+ // *see note in addExit()
+ BlockId newBlock();
+
+ // Adds a new block, but does not mark it current or add an exit from the current block
+ BlockId newIsolatedBlock();
+
+ // Adds an exit from the 'from' block to the 'to' block
+ // Note that we skip adding the exit if the 'from' block is itself unreachable; this means that
+ // we don't actually have to trace the tree to see if a particular block is unreachable, we can
+ // just check to see if it has any entrances. This does require a bit of care in the order in
+ // which we set the CFG up.
+ void addExit(BlockId from, BlockId to);
+
+ friend class CFGGenerator;
+};
+
+/**
+ * Converts functions into control flow graphs.
+ */
+class CFGGenerator {
+public:
+ CFGGenerator() {}
+
+ CFG getCFG(const FunctionDefinition& f);
+
+private:
+ void addStatement(CFG& cfg, const Statement* s);
+
+ void addExpression(CFG& cfg, const Expression* e);
+
+ void addLValue(CFG& cfg, const Expression* e);
+
+ std::stack<BlockId> fLoopContinues;
+ std::stack<BlockId> fLoopExits;
+};
+
+}
+
+#endif
« no previous file with comments | « src/sksl/README ('k') | src/sksl/SkSLCFGGenerator.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698