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

Side by Side Diff: src/builtins/x87/builtins-x87.cc

Issue 2372113004: [turbofan] JSGenericLowering mostly uses builtins instead of code stubs now (Closed)
Patch Set: Ross' comments Created 4 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 unified diff | Download patch
« no previous file with comments | « src/builtins/x64/builtins-x64.cc ('k') | src/code-factory.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #if V8_TARGET_ARCH_X87 5 #if V8_TARGET_ARCH_X87
6 6
7 #include "src/code-factory.h" 7 #include "src/code-factory.h"
8 #include "src/codegen.h" 8 #include "src/codegen.h"
9 #include "src/deoptimizer.h" 9 #include "src/deoptimizer.h"
10 #include "src/full-codegen/full-codegen.h" 10 #include "src/full-codegen/full-codegen.h"
(...skipping 2491 matching lines...) Expand 10 before | Expand all | Expand 10 after
2502 { 2502 {
2503 // Convert receiver using ToObject. 2503 // Convert receiver using ToObject.
2504 // TODO(bmeurer): Inline the allocation here to avoid building the frame 2504 // TODO(bmeurer): Inline the allocation here to avoid building the frame
2505 // in the fast case? (fall back to AllocateInNewSpace?) 2505 // in the fast case? (fall back to AllocateInNewSpace?)
2506 FrameScope scope(masm, StackFrame::INTERNAL); 2506 FrameScope scope(masm, StackFrame::INTERNAL);
2507 __ SmiTag(eax); 2507 __ SmiTag(eax);
2508 __ Push(eax); 2508 __ Push(eax);
2509 __ Push(edi); 2509 __ Push(edi);
2510 __ mov(eax, ecx); 2510 __ mov(eax, ecx);
2511 __ Push(esi); 2511 __ Push(esi);
2512 ToObjectStub stub(masm->isolate()); 2512 __ Call(masm->isolate()->builtins()->ToObject(),
2513 __ CallStub(&stub); 2513 RelocInfo::CODE_TARGET);
2514 __ Pop(esi); 2514 __ Pop(esi);
2515 __ mov(ecx, eax); 2515 __ mov(ecx, eax);
2516 __ Pop(edi); 2516 __ Pop(edi);
2517 __ Pop(eax); 2517 __ Pop(eax);
2518 __ SmiUntag(eax); 2518 __ SmiUntag(eax);
2519 } 2519 }
2520 __ mov(edx, FieldOperand(edi, JSFunction::kSharedFunctionInfoOffset)); 2520 __ mov(edx, FieldOperand(edi, JSFunction::kSharedFunctionInfoOffset));
2521 __ bind(&convert_receiver); 2521 __ bind(&convert_receiver);
2522 } 2522 }
2523 __ mov(Operand(esp, eax, times_pointer_size, kPointerSize), ecx); 2523 __ mov(Operand(esp, eax, times_pointer_size, kPointerSize), ecx);
(...skipping 642 matching lines...) Expand 10 before | Expand all | Expand 10 after
3166 3166
3167 void Builtins::Generate_InterpreterOnStackReplacement(MacroAssembler* masm) { 3167 void Builtins::Generate_InterpreterOnStackReplacement(MacroAssembler* masm) {
3168 Generate_OnStackReplacementHelper(masm, true); 3168 Generate_OnStackReplacementHelper(masm, true);
3169 } 3169 }
3170 3170
3171 #undef __ 3171 #undef __
3172 } // namespace internal 3172 } // namespace internal
3173 } // namespace v8 3173 } // namespace v8
3174 3174
3175 #endif // V8_TARGET_ARCH_X87 3175 #endif // V8_TARGET_ARCH_X87
OLDNEW
« no previous file with comments | « src/builtins/x64/builtins-x64.cc ('k') | src/code-factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698