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

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

Issue 1604923002: [Interpreter] Always store current context in the frames context slot. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@int_fix_block_context_scope
Patch Set: Rebase Created 4 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 unified diff | Download patch
« no previous file with comments | « src/ppc/builtins-ppc.cc ('k') | src/x87/builtins-x87.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_X64 5 #if V8_TARGET_ARCH_X64
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 803 matching lines...) Expand 10 before | Expand all | Expand 10 after
814 // Initialize register file register and dispatch table register. 814 // Initialize register file register and dispatch table register.
815 __ movp(kInterpreterRegisterFileRegister, rbp); 815 __ movp(kInterpreterRegisterFileRegister, rbp);
816 __ addp(kInterpreterRegisterFileRegister, 816 __ addp(kInterpreterRegisterFileRegister,
817 Immediate(InterpreterFrameConstants::kRegisterFilePointerFromFp)); 817 Immediate(InterpreterFrameConstants::kRegisterFilePointerFromFp));
818 __ LoadRoot(kInterpreterDispatchTableRegister, 818 __ LoadRoot(kInterpreterDispatchTableRegister,
819 Heap::kInterpreterTableRootIndex); 819 Heap::kInterpreterTableRootIndex);
820 __ addp(kInterpreterDispatchTableRegister, 820 __ addp(kInterpreterDispatchTableRegister,
821 Immediate(FixedArray::kHeaderSize - kHeapObjectTag)); 821 Immediate(FixedArray::kHeaderSize - kHeapObjectTag));
822 822
823 // Get the context from the frame. 823 // Get the context from the frame.
824 // TODO(rmcilroy): Update interpreter frame to expect current context at the
825 // context slot instead of the function context.
826 __ movp(kContextRegister, 824 __ movp(kContextRegister,
827 Operand(kInterpreterRegisterFileRegister, 825 Operand(kInterpreterRegisterFileRegister,
828 InterpreterFrameConstants::kContextFromRegisterPointer)); 826 InterpreterFrameConstants::kContextFromRegisterPointer));
829 827
830 // Get the bytecode array pointer from the frame. 828 // Get the bytecode array pointer from the frame.
831 __ movp(rbx, 829 __ movp(rbx,
832 Operand(kInterpreterRegisterFileRegister, 830 Operand(kInterpreterRegisterFileRegister,
833 InterpreterFrameConstants::kFunctionFromRegisterPointer)); 831 InterpreterFrameConstants::kFunctionFromRegisterPointer));
834 __ movp(rbx, FieldOperand(rbx, JSFunction::kSharedFunctionInfoOffset)); 832 __ movp(rbx, FieldOperand(rbx, JSFunction::kSharedFunctionInfoOffset));
835 __ movp(kInterpreterBytecodeArrayRegister, 833 __ movp(kInterpreterBytecodeArrayRegister,
(...skipping 1785 matching lines...) Expand 10 before | Expand all | Expand 10 after
2621 __ ret(0); 2619 __ ret(0);
2622 } 2620 }
2623 2621
2624 2622
2625 #undef __ 2623 #undef __
2626 2624
2627 } // namespace internal 2625 } // namespace internal
2628 } // namespace v8 2626 } // namespace v8
2629 2627
2630 #endif // V8_TARGET_ARCH_X64 2628 #endif // V8_TARGET_ARCH_X64
OLDNEW
« no previous file with comments | « src/ppc/builtins-ppc.cc ('k') | src/x87/builtins-x87.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698