X87: Support for super keyed loads where key is a name.
port r24403. original commit message: Support for super keyed loads where key is a name. BUG= R=weiliang.lin@intel.com Review URL: https://codereview.chromium.org/637203002 Patch from Chunyang Dai <chunyang.dai@intel.com>. git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24449 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
This commit is contained in:
parent
5c0bd0cb00
commit
956591293c
@ -2287,6 +2287,14 @@ void FullCodeGenerator::EmitKeyedPropertyLoad(Property* prop) {
|
||||
}
|
||||
|
||||
|
||||
void FullCodeGenerator::EmitKeyedSuperPropertyLoad(Property* prop) {
|
||||
// Stack: receiver, home_object, key.
|
||||
SetSourcePosition(prop->position());
|
||||
|
||||
__ CallRuntime(Runtime::kLoadKeyedFromSuper, 3);
|
||||
}
|
||||
|
||||
|
||||
void FullCodeGenerator::EmitInlineSmiBinaryOp(BinaryOperation* expr,
|
||||
Token::Value op,
|
||||
OverwriteMode mode,
|
||||
@ -2592,11 +2600,19 @@ void FullCodeGenerator::VisitProperty(Property* expr) {
|
||||
PrepareForBailoutForId(expr->LoadId(), TOS_REG);
|
||||
context()->Plug(eax);
|
||||
} else {
|
||||
VisitForStackValue(expr->obj());
|
||||
VisitForAccumulatorValue(expr->key());
|
||||
__ pop(LoadDescriptor::ReceiverRegister()); // Object.
|
||||
__ Move(LoadDescriptor::NameRegister(), result_register()); // Key.
|
||||
EmitKeyedPropertyLoad(expr);
|
||||
if (!expr->IsSuperAccess()) {
|
||||
VisitForStackValue(expr->obj());
|
||||
VisitForAccumulatorValue(expr->key());
|
||||
__ pop(LoadDescriptor::ReceiverRegister()); // Object.
|
||||
__ Move(LoadDescriptor::NameRegister(), result_register()); // Key.
|
||||
EmitKeyedPropertyLoad(expr);
|
||||
} else {
|
||||
VisitForStackValue(expr->obj()->AsSuperReference()->this_var());
|
||||
EmitLoadHomeObject(expr->obj()->AsSuperReference());
|
||||
__ push(result_register());
|
||||
VisitForStackValue(expr->key());
|
||||
EmitKeyedSuperPropertyLoad(expr);
|
||||
}
|
||||
context()->Plug(eax);
|
||||
}
|
||||
}
|
||||
@ -2699,6 +2715,40 @@ void FullCodeGenerator::EmitKeyedCallWithLoadIC(Call* expr,
|
||||
}
|
||||
|
||||
|
||||
void FullCodeGenerator::EmitKeyedSuperCallWithLoadIC(Call* expr) {
|
||||
Expression* callee = expr->expression();
|
||||
DCHECK(callee->IsProperty());
|
||||
Property* prop = callee->AsProperty();
|
||||
DCHECK(prop->IsSuperAccess());
|
||||
|
||||
SetSourcePosition(prop->position());
|
||||
// Load the function from the receiver.
|
||||
SuperReference* super_ref = callee->AsProperty()->obj()->AsSuperReference();
|
||||
EmitLoadHomeObject(super_ref);
|
||||
__ push(eax);
|
||||
VisitForAccumulatorValue(super_ref->this_var());
|
||||
__ push(eax);
|
||||
__ push(eax);
|
||||
__ push(Operand(esp, kPointerSize * 2));
|
||||
VisitForStackValue(prop->key());
|
||||
// Stack here:
|
||||
// - home_object
|
||||
// - this (receiver)
|
||||
// - this (receiver) <-- LoadKeyedFromSuper will pop here and below.
|
||||
// - home_object
|
||||
// - key
|
||||
__ CallRuntime(Runtime::kLoadKeyedFromSuper, 3);
|
||||
|
||||
// Replace home_object with target function.
|
||||
__ mov(Operand(esp, kPointerSize), eax);
|
||||
|
||||
// Stack here:
|
||||
// - target function
|
||||
// - this (receiver)
|
||||
EmitCall(expr, CallICState::METHOD);
|
||||
}
|
||||
|
||||
|
||||
void FullCodeGenerator::EmitCall(Call* expr, CallICState::CallType call_type) {
|
||||
// Load the arguments.
|
||||
ZoneList<Expression*>* args = expr->arguments();
|
||||
@ -2840,9 +2890,12 @@ void FullCodeGenerator::VisitCall(Call* expr) {
|
||||
} else if (call_type == Call::PROPERTY_CALL) {
|
||||
Property* property = callee->AsProperty();
|
||||
bool is_named_call = property->key()->IsPropertyName();
|
||||
// super.x() is handled in EmitCallWithLoadIC.
|
||||
if (property->IsSuperAccess() && is_named_call) {
|
||||
EmitSuperCallWithLoadIC(expr);
|
||||
if (property->IsSuperAccess()) {
|
||||
if (is_named_call) {
|
||||
EmitSuperCallWithLoadIC(expr);
|
||||
} else {
|
||||
EmitKeyedSuperCallWithLoadIC(expr);
|
||||
}
|
||||
} else {
|
||||
{
|
||||
PreservePositionScope scope(masm()->positions_recorder());
|
||||
|
Loading…
Reference in New Issue
Block a user