Reviewers: jarin,
Description:
[turbofan] Remove explicit context specialization from inlining.
Ideally inliner itself should not deal with context specialization at
all, since this is all handled in the pipeline instead (actually
inlining already runs together with context specialization), and the
inlining logic should not care about the specialization mode.
[email protected]
Please review this at https://codereview.chromium.org/1217973003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+0, -5 lines):
M src/compiler/js-inlining.cc
Index: src/compiler/js-inlining.cc
diff --git a/src/compiler/js-inlining.cc b/src/compiler/js-inlining.cc
index
1d7a9b755e48a1271b335f228b0dba4c9716066c..6d7de1d30df21621c02dda5590a722d387eb5dff
100644
--- a/src/compiler/js-inlining.cc
+++ b/src/compiler/js-inlining.cc
@@ -9,7 +9,6 @@
#include "src/compiler/all-nodes.h"
#include "src/compiler/ast-graph-builder.h"
#include "src/compiler/common-operator.h"
-#include "src/compiler/js-context-specialization.h"
#include "src/compiler/js-operator.h"
#include "src/compiler/node-matchers.h"
#include "src/compiler/node-properties.h"
@@ -282,10 +281,6 @@ Reduction JSInliner::Reduce(Node* node) {
// type feedback in the compiler.
AstGraphBuilder graph_builder(local_zone_, &info, &jsgraph);
graph_builder.CreateGraph(true, false);
- GraphReducer graph_reducer(local_zone_, &graph);
- JSContextSpecializer context_specializer(&graph_reducer, &jsgraph);
- graph_reducer.AddReducer(&context_specializer);
- graph_reducer.ReduceGraph();
CopyVisitor visitor(&graph, jsgraph_->graph(), info.zone());
visitor.CopyGraph();
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
