Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: src/builtins/x87/builtins-x87.cc

Issue 2646343002: X87: [Turbofan] Implement super calls with spread bytecode in assembly code. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/x87/builtins-x87.cc
diff --git a/src/builtins/x87/builtins-x87.cc b/src/builtins/x87/builtins-x87.cc
index d635fdcd494fdeefb22b6acdc5071a5c6a234275..6f92b3fc0f351c30cb1dc8262e04fbab4f97edbe 100644
--- a/src/builtins/x87/builtins-x87.cc
+++ b/src/builtins/x87/builtins-x87.cc
@@ -2825,6 +2825,191 @@ void Builtins::Generate_Construct(MacroAssembler* masm) {
}
// static
+void Builtins::Generate_ConstructWithSpread(MacroAssembler* masm) {
+ // ----------- S t a t e -------------
+ // -- eax : the number of arguments (not including the receiver)
+ // -- edx : the new target (either the same as the constructor or
+ // the JSFunction on which new was invoked initially)
+ // -- edi : the constructor to call (can be any Object)
+ // -----------------------------------
+
+ // Free up some registers.
+ // Save edx/edi to stX0/stX1.
+ __ push(edx);
+ __ push(edi);
+ __ fld_s(MemOperand(esp, 0));
+ __ fld_s(MemOperand(esp, 4));
+ __ lea(esp, Operand(esp, 2 * kFloatSize));
+
+ Register argc = eax;
+
+ Register scratch = ecx;
+ Register scratch2 = edi;
+
+ Register spread = ebx;
+ Register spread_map = edx;
+
+ __ mov(spread, Operand(esp, kPointerSize));
+ __ mov(spread_map, FieldOperand(spread, HeapObject::kMapOffset));
+
+ Label runtime_call, push_args;
+ // Check that the spread is an array.
+ __ CmpInstanceType(spread_map, JS_ARRAY_TYPE);
+ __ j(not_equal, &runtime_call);
+
+ // Check that we have the original ArrayPrototype.
+ __ mov(scratch, FieldOperand(spread_map, Map::kPrototypeOffset));
+ __ mov(scratch2, NativeContextOperand());
+ __ cmp(scratch,
+ ContextOperand(scratch2, Context::INITIAL_ARRAY_PROTOTYPE_INDEX));
+ __ j(not_equal, &runtime_call);
+
+ // Check that the ArrayPrototype hasn't been modified in a way that would
+ // affect iteration.
+ __ LoadRoot(scratch, Heap::kArrayIteratorProtectorRootIndex);
+ __ cmp(FieldOperand(scratch, Cell::kValueOffset),
+ Immediate(Smi::FromInt(Isolate::kProtectorValid)));
+ __ j(not_equal, &runtime_call);
+
+ // Check that the map of the initial array iterator hasn't changed.
+ __ mov(scratch2, NativeContextOperand());
+ __ mov(scratch,
+ ContextOperand(scratch2,
+ Context::INITIAL_ARRAY_ITERATOR_PROTOTYPE_INDEX));
+ __ mov(scratch, FieldOperand(scratch, HeapObject::kMapOffset));
+ __ cmp(scratch,
+ ContextOperand(scratch2,
+ Context::INITIAL_ARRAY_ITERATOR_PROTOTYPE_MAP_INDEX));
+ __ j(not_equal, &runtime_call);
+
+ // For FastPacked kinds, iteration will have the same effect as simply
+ // accessing each property in order.
+ Label no_protector_check;
+ __ mov(scratch, FieldOperand(spread_map, Map::kBitField2Offset));
+ __ DecodeField<Map::ElementsKindBits>(scratch);
+ __ cmp(scratch, Immediate(LAST_FAST_ELEMENTS_KIND));
+ __ j(above, &runtime_call);
+ // For non-FastHoley kinds, we can skip the protector check.
+ __ cmp(scratch, Immediate(FAST_SMI_ELEMENTS));
+ __ j(equal, &no_protector_check);
+ __ cmp(scratch, Immediate(FAST_ELEMENTS));
+ __ j(equal, &no_protector_check);
+ __ cmp(scratch, Immediate(FAST_DOUBLE_ELEMENTS));
+ __ j(equal, &no_protector_check);
+ // Check the ArrayProtector cell.
+ __ LoadRoot(scratch, Heap::kArrayProtectorRootIndex);
+ __ cmp(FieldOperand(scratch, PropertyCell::kValueOffset),
+ Immediate(Smi::FromInt(Isolate::kProtectorValid)));
+ __ j(not_equal, &runtime_call);
+
+ __ bind(&no_protector_check);
+ // Load the FixedArray backing store.
+ __ mov(spread, FieldOperand(spread, JSArray::kElementsOffset));
+ // Free up some registers.
+ __ jmp(&push_args);
+
+ __ bind(&runtime_call);
+ {
+ // Call the builtin for the result of the spread.
+ FrameScope scope(masm, StackFrame::INTERNAL);
+ // Need to save these on the stack.
+ // Restore edx/edi from stX0/stX1.
+ __ lea(esp, Operand(esp, -2 * kFloatSize));
+ __ fstp_s(MemOperand(esp, 0));
+ __ fstp_s(MemOperand(esp, 4));
+ __ pop(edx);
+ __ pop(edi);
+
+ __ Push(edi);
+ __ Push(edx);
+ __ SmiTag(argc);
+ __ Push(argc);
+ __ Push(spread);
+ __ CallRuntime(Runtime::kSpreadIterableFixed);
+ __ mov(spread, eax);
+ __ Pop(argc);
+ __ SmiUntag(argc);
+ __ Pop(edx);
+ __ Pop(edi);
+ // Free up some registers.
+ // Save edx/edi to stX0/stX1.
+ __ push(edx);
+ __ push(edi);
+ __ fld_s(MemOperand(esp, 0));
+ __ fld_s(MemOperand(esp, 4));
+ __ lea(esp, Operand(esp, 2 * kFloatSize));
+ }
+
+ Register spread_len = edx;
+ Register return_address = edi;
+ __ bind(&push_args);
+ {
+ // Pop the return address and spread argument.
+ __ PopReturnAddressTo(return_address);
+ __ Pop(scratch);
+
+ // Calculate the new nargs including the result of the spread.
+ __ mov(spread_len, FieldOperand(spread, FixedArray::kLengthOffset));
+ __ SmiUntag(spread_len);
+ // argc += spread_len - 1. Subtract 1 for the spread itself.
+ __ lea(argc, Operand(argc, spread_len, times_1, -1));
+ }
+
+ // Check for stack overflow.
+ {
+ // Check the stack for overflow. We are not trying to catch interruptions
+ // (i.e. debug break and preemption) here, so check the "real stack limit".
+ Label done;
+ __ LoadRoot(scratch, Heap::kRealStackLimitRootIndex);
+ // Make scratch the space we have left. The stack might already be
+ // overflowed here which will cause scratch to become negative.
+ __ neg(scratch);
+ __ add(scratch, esp);
+ __ sar(scratch, kPointerSizeLog2);
+ // Check if the arguments will overflow the stack.
+ __ cmp(scratch, spread_len);
+ __ j(greater, &done, Label::kNear); // Signed comparison.
+ __ TailCallRuntime(Runtime::kThrowStackOverflow);
+ __ bind(&done);
+ }
+
+ // Put the evaluated spread onto the stack as additional arguments.
+ {
+ Register scratch2 = esi;
+ // __ movd(xmm2, esi);
+ // Save esi to stX0, edx/edi in stX1/stX2 now.
+ __ push(esi);
+ __ fld_s(MemOperand(esp, 0));
+ __ lea(esp, Operand(esp, 1 * kFloatSize));
+
+ __ mov(scratch, Immediate(0));
+ Label done, loop;
+ __ bind(&loop);
+ __ cmp(scratch, spread_len);
+ __ j(equal, &done, Label::kNear);
+ __ mov(scratch2, FieldOperand(spread, scratch, times_pointer_size,
+ FixedArray::kHeaderSize));
+ __ Push(scratch2);
+ __ inc(scratch);
+ __ jmp(&loop);
+ __ bind(&done);
+ __ PushReturnAddressFrom(return_address);
+
+ // Now Restore esi from stX0, edx/edi from stX1/stX2.
+ __ lea(esp, Operand(esp, -3 * kFloatSize));
+ __ fstp_s(MemOperand(esp, 0));
+ __ fstp_s(MemOperand(esp, 4));
+ __ fstp_s(MemOperand(esp, 8));
+ __ pop(esi);
+ __ pop(edx);
+ __ pop(edi);
+ }
+
+ // Dispatch.
+ __ Jump(masm->isolate()->builtins()->Construct(), RelocInfo::CODE_TARGET);
+}
+
+// static
void Builtins::Generate_AllocateInNewSpace(MacroAssembler* masm) {
// ----------- S t a t e -------------
// -- edx : requested object size (untagged)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698