Index: src/arm64/full-codegen-arm64.cc |
diff --git a/src/arm64/full-codegen-arm64.cc b/src/arm64/full-codegen-arm64.cc |
index eeaef5f2962690b11b090401b834da73c98469d3..e3bd9d9c23274c9093e19824bfcb17173c253390 100644 |
--- a/src/arm64/full-codegen-arm64.cc |
+++ b/src/arm64/full-codegen-arm64.cc |
@@ -1862,22 +1862,8 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
Comment cmnt(masm_, "[ Assignment"); |
- // Left-hand side can only be a property, a global or a (parameter or local) |
- // slot. |
- enum LhsKind { |
- VARIABLE, |
- NAMED_PROPERTY, |
- KEYED_PROPERTY, |
- NAMED_SUPER_PROPERTY |
- }; |
- LhsKind assign_type = VARIABLE; |
Property* property = expr->target()->AsProperty(); |
- if (property != NULL) { |
- assign_type = (property->key()->IsPropertyName()) |
- ? (property->IsSuperAccess() ? NAMED_SUPER_PROPERTY |
- : NAMED_PROPERTY) |
- : KEYED_PROPERTY; |
- } |
+ LhsKind assign_type = GetAssignType(property); |
// Evaluate LHS expression. |
switch (assign_type) { |
@@ -1903,6 +1889,20 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
__ Push(scratch, result_register()); |
} |
break; |
+ case KEYED_SUPER_PROPERTY: |
+ VisitForStackValue(property->obj()->AsSuperReference()->this_var()); |
+ EmitLoadHomeObject(property->obj()->AsSuperReference()); |
+ __ Push(result_register()); |
+ VisitForAccumulatorValue(property->key()); |
+ __ Push(result_register()); |
+ if (expr->is_compound()) { |
+ const Register scratch1 = x10; |
+ const Register scratch2 = x11; |
+ __ Peek(scratch1, 2 * kPointerSize); |
+ __ Peek(scratch2, kPointerSize); |
+ __ Push(scratch1, scratch2, result_register()); |
+ } |
+ break; |
case KEYED_PROPERTY: |
if (expr->is_compound()) { |
VisitForStackValue(property->obj()); |
@@ -1933,6 +1933,10 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
EmitNamedSuperPropertyLoad(property); |
PrepareForBailoutForId(property->LoadId(), TOS_REG); |
break; |
+ case KEYED_SUPER_PROPERTY: |
+ EmitKeyedSuperPropertyLoad(property); |
+ PrepareForBailoutForId(property->LoadId(), TOS_REG); |
+ break; |
case KEYED_PROPERTY: |
EmitKeyedPropertyLoad(property); |
PrepareForBailoutForId(property->LoadId(), TOS_REG); |
@@ -1983,6 +1987,10 @@ void FullCodeGenerator::VisitAssignment(Assignment* expr) { |
EmitNamedSuperPropertyStore(property); |
context()->Plug(x0); |
break; |
+ case KEYED_SUPER_PROPERTY: |
+ EmitKeyedSuperPropertyStore(property); |
+ context()->Plug(x0); |
+ break; |
case KEYED_PROPERTY: |
EmitKeyedPropertyAssignment(expr); |
break; |
@@ -2314,14 +2322,27 @@ void FullCodeGenerator::EmitNamedSuperPropertyStore(Property* prop) { |
Literal* key = prop->key()->AsLiteral(); |
DCHECK(key != NULL); |
- __ Push(x0); |
__ Push(key->value()); |
+ __ Push(x0); |
__ CallRuntime((strict_mode() == STRICT ? Runtime::kStoreToSuper_Strict |
: Runtime::kStoreToSuper_Sloppy), |
4); |
} |
+void FullCodeGenerator::EmitKeyedSuperPropertyStore(Property* prop) { |
+ // Assignment to named property of super. |
+ // x0 : value |
+ // stack : receiver ('this'), home_object, key |
+ DCHECK(prop != NULL); |
+ |
+ __ Push(x0); |
+ __ CallRuntime((strict_mode() == STRICT ? Runtime::kStoreKeyedToSuper_Strict |
+ : Runtime::kStoreKeyedToSuper_Sloppy), |
+ 4); |
+} |
+ |
+ |
void FullCodeGenerator::EmitKeyedPropertyAssignment(Assignment* expr) { |
ASM_LOCATION("FullCodeGenerator::EmitKeyedPropertyAssignment"); |
// Assignment to a property, using a keyed store IC. |
@@ -4095,24 +4116,8 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
Comment cmnt(masm_, "[ CountOperation"); |
SetSourcePosition(expr->position()); |
- // Expression can only be a property, a global or a (parameter or local) |
- // slot. |
- enum LhsKind { |
- VARIABLE, |
- NAMED_PROPERTY, |
- KEYED_PROPERTY, |
- NAMED_SUPER_PROPERTY |
- }; |
- LhsKind assign_type = VARIABLE; |
Property* prop = expr->expression()->AsProperty(); |
- // In case of a property we use the uninitialized expression context |
- // of the key to detect a named property. |
- if (prop != NULL) { |
- assign_type = |
- (prop->key()->IsPropertyName()) |
- ? (prop->IsSuperAccess() ? NAMED_SUPER_PROPERTY : NAMED_PROPERTY) |
- : KEYED_PROPERTY; |
- } |
+ LhsKind assign_type = GetAssignType(prop); |
// Evaluate expression and get value. |
if (assign_type == VARIABLE) { |
@@ -4124,26 +4129,52 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
if (expr->is_postfix() && !context()->IsEffect()) { |
__ Push(xzr); |
} |
- if (assign_type == NAMED_PROPERTY) { |
- // Put the object both on the stack and in the register. |
- VisitForStackValue(prop->obj()); |
- __ Peek(LoadDescriptor::ReceiverRegister(), 0); |
- EmitNamedPropertyLoad(prop); |
- } else if (assign_type == NAMED_SUPER_PROPERTY) { |
- VisitForStackValue(prop->obj()->AsSuperReference()->this_var()); |
- EmitLoadHomeObject(prop->obj()->AsSuperReference()); |
- __ Push(result_register()); |
- const Register scratch = x10; |
- __ Peek(scratch, kPointerSize); |
- __ Push(scratch, result_register()); |
- EmitNamedSuperPropertyLoad(prop); |
- } else { |
- // KEYED_PROPERTY |
- VisitForStackValue(prop->obj()); |
- VisitForStackValue(prop->key()); |
- __ Peek(LoadDescriptor::ReceiverRegister(), 1 * kPointerSize); |
- __ Peek(LoadDescriptor::NameRegister(), 0); |
- EmitKeyedPropertyLoad(prop); |
+ switch (assign_type) { |
+ case NAMED_PROPERTY: { |
+ // Put the object both on the stack and in the register. |
+ VisitForStackValue(prop->obj()); |
+ __ Peek(LoadDescriptor::ReceiverRegister(), 0); |
+ EmitNamedPropertyLoad(prop); |
+ break; |
+ } |
+ |
+ case NAMED_SUPER_PROPERTY: { |
+ VisitForStackValue(prop->obj()->AsSuperReference()->this_var()); |
+ EmitLoadHomeObject(prop->obj()->AsSuperReference()); |
+ __ Push(result_register()); |
+ const Register scratch = x10; |
+ __ Peek(scratch, kPointerSize); |
+ __ Push(scratch, result_register()); |
+ EmitNamedSuperPropertyLoad(prop); |
+ break; |
+ } |
+ |
+ case KEYED_SUPER_PROPERTY: { |
+ VisitForStackValue(prop->obj()->AsSuperReference()->this_var()); |
+ EmitLoadHomeObject(prop->obj()->AsSuperReference()); |
+ __ Push(result_register()); |
+ VisitForAccumulatorValue(prop->key()); |
+ __ Push(result_register()); |
+ const Register scratch1 = x10; |
+ const Register scratch2 = x11; |
+ __ Peek(scratch1, 2 * kPointerSize); |
+ __ Peek(scratch2, kPointerSize); |
+ __ Push(scratch1, scratch2, result_register()); |
+ EmitKeyedSuperPropertyLoad(prop); |
+ break; |
+ } |
+ |
+ case KEYED_PROPERTY: { |
+ VisitForStackValue(prop->obj()); |
+ VisitForStackValue(prop->key()); |
+ __ Peek(LoadDescriptor::ReceiverRegister(), 1 * kPointerSize); |
+ __ Peek(LoadDescriptor::NameRegister(), 0); |
+ EmitKeyedPropertyLoad(prop); |
+ break; |
+ } |
+ |
+ case VARIABLE: |
+ UNREACHABLE(); |
} |
} |
@@ -4183,6 +4214,9 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
case KEYED_PROPERTY: |
__ Poke(x0, kPointerSize * 2); |
break; |
+ case KEYED_SUPER_PROPERTY: |
+ __ Poke(x0, kPointerSize * 3); |
+ break; |
} |
} |
} |
@@ -4216,6 +4250,9 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
case KEYED_PROPERTY: |
__ Poke(x0, 2 * kXRegSize); |
break; |
+ case KEYED_SUPER_PROPERTY: |
+ __ Poke(x0, 3 * kXRegSize); |
+ break; |
} |
} |
} |
@@ -4284,6 +4321,17 @@ void FullCodeGenerator::VisitCountOperation(CountOperation* expr) { |
} |
break; |
} |
+ case KEYED_SUPER_PROPERTY: { |
+ EmitKeyedSuperPropertyStore(prop); |
+ if (expr->is_postfix()) { |
+ if (!context()->IsEffect()) { |
+ context()->PlugTOS(); |
+ } |
+ } else { |
+ context()->Plug(x0); |
+ } |
+ break; |
+ } |
case KEYED_PROPERTY: { |
__ Pop(StoreDescriptor::NameRegister()); |
__ Pop(StoreDescriptor::ReceiverRegister()); |