Index: src/ia32/full-codegen-ia32.cc |
diff --git a/src/ia32/full-codegen-ia32.cc b/src/ia32/full-codegen-ia32.cc |
index e538ee4b9f9484277c2180c11ee2bd9c23d865f3..1f7095f578082e717a05c9ecd2cb5e121e5be613 100644 |
--- a/src/ia32/full-codegen-ia32.cc |
+++ b/src/ia32/full-codegen-ia32.cc |
@@ -911,9 +911,7 @@ void FullCodeGenerator::VisitForInStatement(ForInStatement* stmt) { |
__ bind(&update_each); |
__ mov(result_register(), ebx); |
// Perform the assignment as if via '='. |
- { EffectContext context(this); |
- EmitAssignment(stmt->each(), stmt->AssignmentId()); |
- } |
+ EmitAssignment(stmt->each()); |
// Generate code for the body of the loop. |
Visit(stmt->body()); |
@@ -1480,7 +1478,7 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
// For property compound assignments we need another deoptimization |
// point after the property load. |
if (property != NULL) { |
- PrepareForBailoutForId(expr->CompoundLoadId(), TOS_REG); |
+ PrepareForBailoutForId(expr->compound_bailout_id(), TOS_REG); |
} |
Token::Value op = expr->binary_op(); |
@@ -1523,8 +1521,6 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
case VARIABLE: |
EmitVariableAssignment(expr->target()->AsVariableProxy()->var(), |
expr->op()); |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
- context()->Plug(eax); |
break; |
case NAMED_PROPERTY: |
EmitNamedPropertyAssignment(expr); |
@@ -1853,7 +1849,7 @@ void FullCodeGenerator::EmitBinaryOp(Token::Value op, |
} |
-void FullCodeGenerator::EmitAssignment(Expression* expr, int bailout_ast_id) { |
+void FullCodeGenerator::EmitAssignment(Expression* expr) { |
// Invalid left-hand sides are rewritten to have a 'throw |
// ReferenceError' on the left-hand side. |
if (!expr->IsValidLeftHandSide()) { |
@@ -1901,8 +1897,6 @@ void FullCodeGenerator::EmitAssignment(Expression* expr, int bailout_ast_id) { |
break; |
} |
} |
- PrepareForBailoutForId(bailout_ast_id, TOS_REG); |
- context()->Plug(eax); |
} |
@@ -1975,6 +1969,8 @@ void FullCodeGenerator::EmitVariableAssignment(Variable* var, |
} |
__ bind(&done); |
} |
+ |
+ context()->Plug(eax); |
} |
@@ -2011,10 +2007,10 @@ void FullCodeGenerator::EmitNamedPropertyAssignment(Assignment* expr) { |
__ push(Operand(esp, kPointerSize)); // Receiver is under value. |
__ CallRuntime(Runtime::kToFastProperties, 1); |
__ pop(eax); |
- __ Drop(1); |
+ context()->DropAndPlug(1, eax); |
+ } else { |
+ context()->Plug(eax); |
} |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
- context()->Plug(eax); |
} |
@@ -2052,7 +2048,6 @@ void FullCodeGenerator::EmitKeyedPropertyAssignment(Assignment* expr) { |
__ pop(eax); |
} |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
context()->Plug(eax); |
} |
@@ -3754,8 +3749,6 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
{ EffectContext context(this); |
EmitVariableAssignment(expr->expression()->AsVariableProxy()->var(), |
Token::ASSIGN); |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
- context.Plug(eax); |
} |
// For all contexts except EffectContext We have the result on |
// top of the stack. |
@@ -3766,8 +3759,6 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
// Perform the assignment as if via '='. |
EmitVariableAssignment(expr->expression()->AsVariableProxy()->var(), |
Token::ASSIGN); |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
- context()->Plug(eax); |
} |
break; |
case NAMED_PROPERTY: { |
@@ -3775,7 +3766,6 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
__ pop(edx); |
Handle<Code> ic(Builtins::builtin(Builtins::StoreIC_Initialize)); |
EmitCallIC(ic, RelocInfo::CODE_TARGET); |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
if (expr->is_postfix()) { |
if (!context()->IsEffect()) { |
context()->PlugTOS(); |
@@ -3790,7 +3780,6 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
__ pop(edx); |
Handle<Code> ic(Builtins::builtin(Builtins::KeyedStoreIC_Initialize)); |
EmitCallIC(ic, RelocInfo::CODE_TARGET); |
- PrepareForBailoutForId(expr->AssignmentId(), TOS_REG); |
if (expr->is_postfix()) { |
// Result is on the stack |
if (!context()->IsEffect()) { |