Commit 57267fef authored by bmeurer's avatar bmeurer Committed by Commit bot

[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.

R=jarin@chromium.org

Review URL: https://codereview.chromium.org/1217973003

Cr-Commit-Position: refs/heads/master@{#29366}
parent 722e2e2b
......@@ -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();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment