Commit 1b3490ad authored by cdai2's avatar cdai2

X87: ES6 computed property names

port 74e38e34.

original commit message:

   This adds support for computed property names, under the flag
   --harmony-computed-property-names, for both object literals and classes.

   This is a revert of the revert, 7d48fd9d.

BUG=
R=weiliang.lin@intel.com

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

Cr-Commit-Position: refs/heads/master@{#26090}
parent 71fd742f
...@@ -1606,11 +1606,13 @@ void FullCodeGenerator::VisitObjectLiteral(ObjectLiteral* expr) { ...@@ -1606,11 +1606,13 @@ void FullCodeGenerator::VisitObjectLiteral(ObjectLiteral* expr) {
expr->CalculateEmitStore(zone()); expr->CalculateEmitStore(zone());
AccessorTable accessor_table(zone()); AccessorTable accessor_table(zone());
for (int i = 0; i < expr->properties()->length(); i++) { int property_index = 0;
ObjectLiteral::Property* property = expr->properties()->at(i); for (; property_index < expr->properties()->length(); property_index++) {
ObjectLiteral::Property* property = expr->properties()->at(property_index);
if (property->is_computed_name()) break;
if (property->IsCompileTimeValue()) continue; if (property->IsCompileTimeValue()) continue;
Literal* key = property->key(); Literal* key = property->key()->AsLiteral();
Expression* value = property->value(); Expression* value = property->value();
if (!result_saved) { if (!result_saved) {
__ push(eax); // Save result on the stack __ push(eax); // Save result on the stack
...@@ -1690,6 +1692,65 @@ void FullCodeGenerator::VisitObjectLiteral(ObjectLiteral* expr) { ...@@ -1690,6 +1692,65 @@ void FullCodeGenerator::VisitObjectLiteral(ObjectLiteral* expr) {
__ CallRuntime(Runtime::kDefineAccessorPropertyUnchecked, 5); __ CallRuntime(Runtime::kDefineAccessorPropertyUnchecked, 5);
} }
// Object literals have two parts. The "static" part on the left contains no
// computed property names, and so we can compute its map ahead of time; see
// runtime.cc::CreateObjectLiteralBoilerplate. The second "dynamic" part
// starts with the first computed property name, and continues with all
// properties to its right. All the code from above initializes the static
// component of the object literal, and arranges for the map of the result to
// reflect the static order in which the keys appear. For the dynamic
// properties, we compile them into a series of "SetOwnProperty" runtime
// calls. This will preserve insertion order.
for (; property_index < expr->properties()->length(); property_index++) {
ObjectLiteral::Property* property = expr->properties()->at(property_index);
Expression* value = property->value();
if (!result_saved) {
__ push(eax); // Save result on the stack
result_saved = true;
}
__ push(Operand(esp, 0)); // Duplicate receiver.
if (property->kind() == ObjectLiteral::Property::PROTOTYPE) {
DCHECK(!property->is_computed_name());
VisitForStackValue(value);
if (property->emit_store()) {
__ CallRuntime(Runtime::kInternalSetPrototype, 2);
} else {
__ Drop(2);
}
} else {
EmitPropertyKey(property);
VisitForStackValue(value);
switch (property->kind()) {
case ObjectLiteral::Property::CONSTANT:
case ObjectLiteral::Property::MATERIALIZED_LITERAL:
case ObjectLiteral::Property::COMPUTED:
if (property->emit_store()) {
__ push(Immediate(Smi::FromInt(NONE)));
__ CallRuntime(Runtime::kDefineDataPropertyUnchecked, 4);
} else {
__ Drop(3);
}
break;
case ObjectLiteral::Property::PROTOTYPE:
UNREACHABLE();
break;
case ObjectLiteral::Property::GETTER:
__ CallRuntime(Runtime::kDefineGetterPropertyUnchecked, 3);
break;
case ObjectLiteral::Property::SETTER:
__ CallRuntime(Runtime::kDefineSetterPropertyUnchecked, 3);
break;
}
}
}
if (expr->has_function()) { if (expr->has_function()) {
DCHECK(result_saved); DCHECK(result_saved);
__ push(Operand(esp, 0)); __ push(Operand(esp, 0));
...@@ -2381,16 +2442,14 @@ void FullCodeGenerator::EmitClassDefineProperties(ClassLiteral* lit) { ...@@ -2381,16 +2442,14 @@ void FullCodeGenerator::EmitClassDefineProperties(ClassLiteral* lit) {
for (int i = 0; i < lit->properties()->length(); i++) { for (int i = 0; i < lit->properties()->length(); i++) {
ObjectLiteral::Property* property = lit->properties()->at(i); ObjectLiteral::Property* property = lit->properties()->at(i);
Literal* key = property->key()->AsLiteral();
Expression* value = property->value(); Expression* value = property->value();
DCHECK(key != NULL);
if (property->is_static()) { if (property->is_static()) {
__ push(Operand(esp, kPointerSize)); // constructor __ push(Operand(esp, kPointerSize)); // constructor
} else { } else {
__ push(Operand(esp, 0)); // prototype __ push(Operand(esp, 0)); // prototype
} }
VisitForStackValue(key); EmitPropertyKey(property);
VisitForStackValue(value); VisitForStackValue(value);
EmitSetHomeObjectIfNeeded(value, 2); EmitSetHomeObjectIfNeeded(value, 2);
...@@ -2403,11 +2462,11 @@ void FullCodeGenerator::EmitClassDefineProperties(ClassLiteral* lit) { ...@@ -2403,11 +2462,11 @@ void FullCodeGenerator::EmitClassDefineProperties(ClassLiteral* lit) {
break; break;
case ObjectLiteral::Property::GETTER: case ObjectLiteral::Property::GETTER:
__ CallRuntime(Runtime::kDefineClassGetter, 3); __ CallRuntime(Runtime::kDefineGetterPropertyUnchecked, 3);
break; break;
case ObjectLiteral::Property::SETTER: case ObjectLiteral::Property::SETTER:
__ CallRuntime(Runtime::kDefineClassSetter, 3); __ CallRuntime(Runtime::kDefineSetterPropertyUnchecked, 3);
break; break;
default: default:
......
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