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

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

Issue 1402153004: Revert of [Interpreter] Support for operator new. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 2 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 | « src/compiler/interpreter-assembler.cc ('k') | src/ia32/interface-descriptors-ia32.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/ia32/builtins-ia32.cc
diff --git a/src/ia32/builtins-ia32.cc b/src/ia32/builtins-ia32.cc
index 16081779aff56d1f7be6e268b45896ba209aa8a6..4156123f28f41d29a9ef398a84736a2d2aad61a4 100644
--- a/src/ia32/builtins-ia32.cc
+++ b/src/ia32/builtins-ia32.cc
@@ -724,24 +724,6 @@
}
-static void Generate_InterpreterPushArgs(MacroAssembler* masm,
- Register array_limit) {
- // ----------- S t a t e -------------
- // -- ebx : Pointer to the last argument in the args array.
- // -- array_limit : Pointer to one before the first argument in the
- // args array.
- // -----------------------------------
- Label loop_header, loop_check;
- __ jmp(&loop_check);
- __ bind(&loop_header);
- __ Push(Operand(ebx, 0));
- __ sub(ebx, Immediate(kPointerSize));
- __ bind(&loop_check);
- __ cmp(ebx, array_limit);
- __ j(greater, &loop_header, Label::kNear);
-}
-
-
// static
void Builtins::Generate_InterpreterPushArgsAndCall(MacroAssembler* masm) {
// ----------- S t a t e -------------
@@ -750,7 +732,6 @@
// arguments should be consecutive above this, in the same order as
// they are to be pushed onto the stack.
// -- edi : the target to call (can be any Object).
- // -----------------------------------
// Pop return address to allow tail-call after pushing arguments.
__ Pop(edx);
@@ -762,7 +743,15 @@
__ neg(ecx);
__ add(ecx, ebx);
- Generate_InterpreterPushArgs(masm, ecx);
+ // Push the arguments.
+ Label loop_header, loop_check;
+ __ jmp(&loop_check);
+ __ bind(&loop_header);
+ __ Push(Operand(ebx, 0));
+ __ sub(ebx, Immediate(kPointerSize));
+ __ bind(&loop_check);
+ __ cmp(ebx, ecx);
+ __ j(greater, &loop_header, Label::kNear);
// Call the target.
__ Push(edx); // Re-push return address.
@@ -770,51 +759,11 @@
}
-// static
-void Builtins::Generate_InterpreterPushArgsAndConstruct(MacroAssembler* masm) {
- // ----------- S t a t e -------------
- // -- eax : the number of arguments (not including the receiver)
- // -- edx : the original constructor
- // -- edi : the constructor
- // -- ebx : the address of the first argument to be pushed. Subsequent
- // arguments should be consecutive above this, in the same order as
- // they are to be pushed onto the stack.
- // -----------------------------------
-
- // Save number of arguments on the stack below where arguments are going
- // to be pushed.
- __ mov(ecx, eax);
- __ neg(ecx);
- __ mov(Operand(esp, ecx, times_pointer_size, -kPointerSize), eax);
- __ mov(eax, ecx);
-
- // Pop return address to allow tail-call after pushing arguments.
- __ Pop(ecx);
-
- // Find the address of the last argument.
- __ shl(eax, kPointerSizeLog2);
- __ add(eax, ebx);
-
- // Push padding for receiver.
- __ Push(Immediate(0));
-
- Generate_InterpreterPushArgs(masm, eax);
-
- // Restore number of arguments from slot on stack.
- __ mov(eax, Operand(esp, -kPointerSize));
-
- // Re-push return address.
- __ Push(ecx);
-
- // Call the constructor with unmodified eax, edi, ebi values.
- __ Jump(masm->isolate()->builtins()->Construct(), RelocInfo::CONSTRUCT_CALL);
-}
-
-
void Builtins::Generate_CompileLazy(MacroAssembler* masm) {
CallRuntimePassFunction(masm, Runtime::kCompileLazy);
GenerateTailCallToReturnedCode(masm);
}
+
static void CallCompileOptimized(MacroAssembler* masm, bool concurrent) {
« no previous file with comments | « src/compiler/interpreter-assembler.cc ('k') | src/ia32/interface-descriptors-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698