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

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

Issue 1426863005: [Interpreter] Ensure we save the BytecodeArray register properly in InterpreterEntryTrampoline buil… (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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 | « no previous file | src/arm64/builtins-arm64.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_ARM 5 #if V8_TARGET_ARCH_ARM
6 6
7 #include "src/codegen.h" 7 #include "src/codegen.h"
8 #include "src/debug/debug.h" 8 #include "src/debug/debug.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 899 matching lines...) Expand 10 before | Expand all | Expand 10 after
910 // receiver with the global proxy when called as functions (without an 910 // receiver with the global proxy when called as functions (without an
911 // explicit receiver object). 911 // explicit receiver object).
912 // - Code aging of the BytecodeArray object. 912 // - Code aging of the BytecodeArray object.
913 913
914 // Perform stack guard check. 914 // Perform stack guard check.
915 { 915 {
916 Label ok; 916 Label ok;
917 __ LoadRoot(ip, Heap::kStackLimitRootIndex); 917 __ LoadRoot(ip, Heap::kStackLimitRootIndex);
918 __ cmp(sp, Operand(ip)); 918 __ cmp(sp, Operand(ip));
919 __ b(hs, &ok); 919 __ b(hs, &ok);
920 __ push(kInterpreterBytecodeArrayRegister);
920 __ CallRuntime(Runtime::kStackGuard, 0); 921 __ CallRuntime(Runtime::kStackGuard, 0);
922 __ pop(kInterpreterBytecodeArrayRegister);
921 __ bind(&ok); 923 __ bind(&ok);
922 } 924 }
923 925
924 // Load accumulator, register file, bytecode offset, dispatch table into 926 // Load accumulator, register file, bytecode offset, dispatch table into
925 // registers. 927 // registers.
926 __ LoadRoot(kInterpreterAccumulatorRegister, Heap::kUndefinedValueRootIndex); 928 __ LoadRoot(kInterpreterAccumulatorRegister, Heap::kUndefinedValueRootIndex);
927 __ sub(kInterpreterRegisterFileRegister, fp, 929 __ sub(kInterpreterRegisterFileRegister, fp,
928 Operand(kPointerSize + StandardFrameConstants::kFixedFrameSizeFromFp)); 930 Operand(kPointerSize + StandardFrameConstants::kFixedFrameSizeFromFp));
929 __ mov(kInterpreterBytecodeOffsetRegister, 931 __ mov(kInterpreterBytecodeOffsetRegister,
930 Operand(BytecodeArray::kHeaderSize - kHeapObjectTag)); 932 Operand(BytecodeArray::kHeaderSize - kHeapObjectTag));
(...skipping 967 matching lines...) Expand 10 before | Expand all | Expand 10 after
1898 } 1900 }
1899 } 1901 }
1900 1902
1901 1903
1902 #undef __ 1904 #undef __
1903 1905
1904 } // namespace internal 1906 } // namespace internal
1905 } // namespace v8 1907 } // namespace v8
1906 1908
1907 #endif // V8_TARGET_ARCH_ARM 1909 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « no previous file | src/arm64/builtins-arm64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698