Index: src/compiler/ast-graph-builder.cc |
diff --git a/src/compiler/ast-graph-builder.cc b/src/compiler/ast-graph-builder.cc |
index b01016cbeec05f47943d6bbdba1d82c0848436c9..ea604ef77adc14fbe857c05e5fc24987ae244a5a 100644 |
--- a/src/compiler/ast-graph-builder.cc |
+++ b/src/compiler/ast-graph-builder.cc |
@@ -305,7 +305,7 @@ Node* AstGraphBuilder::GetFunctionClosureForContext() { |
// calling eval, not the anonymous closure containing the eval code. |
const Operator* op = |
javascript()->LoadContext(0, Context::CLOSURE_INDEX, false); |
- return NewNode(op, current_context()); |
+ return NewNode(op); |
} else { |
DCHECK(closure_scope->is_function_scope()); |
return GetFunctionClosure(); |
@@ -945,7 +945,7 @@ void AstGraphBuilder::VisitVariableDeclaration(VariableDeclaration* decl) { |
if (variable->binding_needs_init()) { |
Node* value = jsgraph()->TheHoleConstant(); |
const Operator* op = javascript()->StoreContext(0, variable->index()); |
- NewNode(op, current_context(), value); |
+ NewNode(op, value); |
} |
break; |
case VariableLocation::LOOKUP: |
@@ -981,7 +981,7 @@ void AstGraphBuilder::VisitFunctionDeclaration(FunctionDeclaration* decl) { |
VisitForValue(decl->fun()); |
Node* value = environment()->Pop(); |
const Operator* op = javascript()->StoreContext(0, variable->index()); |
- NewNode(op, current_context(), value); |
+ NewNode(op, value); |
break; |
} |
case VariableLocation::LOOKUP: |
@@ -2650,7 +2650,8 @@ Node* AstGraphBuilder::BuildLocalActivationContext(Node* context) { |
Variable* variable = scope->receiver(); |
DCHECK_EQ(0, scope->ContextChainLength(variable->scope())); |
const Operator* op = javascript()->StoreContext(0, variable->index()); |
- NewNode(op, local_context, receiver); |
+ Node* node = NewNode(op, receiver); |
+ NodeProperties::ReplaceContextInput(node, local_context); |
} |
// Copy parameters into context if necessary. |
@@ -2662,7 +2663,8 @@ Node* AstGraphBuilder::BuildLocalActivationContext(Node* context) { |
// Context variable (at bottom of the context chain). |
DCHECK_EQ(0, scope->ContextChainLength(variable->scope())); |
const Operator* op = javascript()->StoreContext(0, variable->index()); |
- NewNode(op, local_context, parameter); |
+ Node* node = NewNode(op, parameter); |
+ NodeProperties::ReplaceContextInput(node, local_context); |
} |
return local_context; |
@@ -2801,7 +2803,7 @@ Node* AstGraphBuilder::BuildVariableLoad(Variable* variable, |
info()->is_function_context_specializing(); |
const Operator* op = |
javascript()->LoadContext(depth, variable->index(), immutable); |
- Node* value = NewNode(op, current_context()); |
+ Node* value = NewNode(op); |
// TODO(titzer): initialization checks are redundant for already |
// initialized immutable context loads, but only specialization knows. |
// Maybe specializer should be a parameter to the graph builder? |
@@ -2918,7 +2920,7 @@ Node* AstGraphBuilder::BuildVariableAssignment( |
// Perform an initialization check for let declared variables. |
const Operator* op = |
javascript()->LoadContext(depth, variable->index(), false); |
- Node* current = NewNode(op, current_context()); |
+ Node* current = NewNode(op); |
value = BuildHoleCheckThenThrow(current, variable, value, bailout_id); |
} else if (mode == CONST && op == Token::INIT) { |
// Perform an initialization check for const {this} variables. |
@@ -2927,7 +2929,7 @@ Node* AstGraphBuilder::BuildVariableAssignment( |
if (variable->is_this()) { |
const Operator* op = |
javascript()->LoadContext(depth, variable->index(), false); |
- Node* current = NewNode(op, current_context()); |
+ Node* current = NewNode(op); |
value = BuildHoleCheckElseThrow(current, variable, value, bailout_id); |
} |
} else if (mode == CONST && op != Token::INIT && |
@@ -2944,14 +2946,14 @@ Node* AstGraphBuilder::BuildVariableAssignment( |
if (variable->binding_needs_init()) { |
const Operator* op = |
javascript()->LoadContext(depth, variable->index(), false); |
- Node* current = NewNode(op, current_context()); |
+ Node* current = NewNode(op); |
BuildHoleCheckThenThrow(current, variable, value, bailout_id); |
} |
// Assignment to const is exception in all modes. |
return BuildThrowConstAssignError(bailout_id); |
} |
const Operator* op = javascript()->StoreContext(depth, variable->index()); |
- return NewNode(op, current_context(), value); |
+ return NewNode(op, value); |
} |
case VariableLocation::LOOKUP: |
case VariableLocation::MODULE: |
@@ -3063,8 +3065,10 @@ Node* AstGraphBuilder::BuildLoadGlobalObject() { |
Node* AstGraphBuilder::BuildLoadNativeContextField(int index) { |
const Operator* op = |
javascript()->LoadContext(0, Context::NATIVE_CONTEXT_INDEX, true); |
- Node* native_context = NewNode(op, current_context()); |
- return NewNode(javascript()->LoadContext(0, index, true), native_context); |
+ Node* native_context = NewNode(op); |
+ Node* result = NewNode(javascript()->LoadContext(0, index, true)); |
+ NodeProperties::ReplaceContextInput(result, native_context); |
+ return result; |
} |
@@ -3499,8 +3503,7 @@ void AstGraphBuilder::Environment::PrepareForOsrEntry() { |
Node* osr_context = effect = contexts()->back(); |
int last = static_cast<int>(contexts()->size() - 1); |
for (int i = last - 1; i >= 0; i--) { |
- osr_context = effect = |
- graph->NewNode(load_op, osr_context, osr_context, effect); |
+ osr_context = effect = graph->NewNode(load_op, osr_context, effect); |
contexts()->at(i) = osr_context; |
} |
UpdateEffectDependency(effect); |