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

Unified Diff: src/compiler/graph-reducer.cc

Issue 1419373012: [turbofan] Run inlining within the regular reducer fixpoint. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Limit inlining depth. Created 5 years, 1 month 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/compiler/graph-reducer.h ('k') | src/compiler/js-inlining-heuristic.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/graph-reducer.cc
diff --git a/src/compiler/graph-reducer.cc b/src/compiler/graph-reducer.cc
index 1be0b6dec7b6ecbfe4d38a3d19f9e7329dc69fdb..6f583d6b6ac4e3f59a2c94ed847c8b600b2f8c75 100644
--- a/src/compiler/graph-reducer.cc
+++ b/src/compiler/graph-reducer.cc
@@ -23,6 +23,9 @@ enum class GraphReducer::State : uint8_t {
};
+void Reducer::Finalize() {}
+
+
GraphReducer::GraphReducer(Zone* zone, Graph* graph, Node* dead)
: graph_(graph),
dead_(dead),
@@ -58,7 +61,11 @@ void GraphReducer::ReduceNode(Node* node) {
Push(node);
}
} else {
- break;
+ // Run all finalizers.
+ for (Reducer* const reducer : reducers_) reducer->Finalize();
+
+ // Check if we have new nodes to revisit.
+ if (revisit_.empty()) break;
}
}
DCHECK(revisit_.empty());
« no previous file with comments | « src/compiler/graph-reducer.h ('k') | src/compiler/js-inlining-heuristic.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698