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

Unified Diff: test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden

Issue 2107193002: [ic] Initialize feedback slots for LoadGlobalIC in Runtime::kDeclareGlobals when possible to ... (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebasing on top of Issue 2127583002 Patch 120001 Created 4 years, 5 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
Index: test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden b/test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden
index 97e403e655b8435819a95fb75dd474a3ce404bcc..9d16d06aff331068af49b9c896df327fdecf4346 100644
--- a/test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden
+++ b/test/cctest/interpreter/bytecode_expectations/DeclareGlobals.golden
@@ -14,13 +14,14 @@ snippet: "
"
frame size: 4
parameter count: 1
-bytecode array length: 31
+bytecode array length: 34
bytecodes: [
B(LdaConstant), U8(0),
B(Star), R(1),
B(LdaZero),
B(Star), R(2),
- B(CallRuntime), U16(Runtime::kDeclareGlobals), R(1), U8(2),
+ B(Mov), R(closure), R(3),
+ B(CallRuntime), U16(Runtime::kDeclareGlobalsForInterpreter), R(1), U8(3),
/* 0 E> */ B(StackCheck),
/* 8 S> */ B(LdaConstant), U8(1),
B(Star), R(1),
@@ -43,15 +44,16 @@ handlers: [
snippet: "
function f() {}
"
-frame size: 2
+frame size: 3
parameter count: 1
-bytecode array length: 15
+bytecode array length: 18
bytecodes: [
B(LdaConstant), U8(0),
B(Star), R(0),
B(LdaZero),
B(Star), R(1),
- B(CallRuntime), U16(Runtime::kDeclareGlobals), R(0), U8(2),
+ B(Mov), R(closure), R(2),
+ B(CallRuntime), U16(Runtime::kDeclareGlobalsForInterpreter), R(0), U8(3),
/* 0 E> */ B(StackCheck),
B(LdaUndefined),
/* 15 S> */ B(Return),
@@ -69,13 +71,14 @@ snippet: "
"
frame size: 4
parameter count: 1
-bytecode array length: 37
+bytecode array length: 40
bytecodes: [
B(LdaConstant), U8(0),
B(Star), R(1),
B(LdaZero),
B(Star), R(2),
- B(CallRuntime), U16(Runtime::kDeclareGlobals), R(1), U8(2),
+ B(Mov), R(closure), R(3),
+ B(CallRuntime), U16(Runtime::kDeclareGlobalsForInterpreter), R(1), U8(3),
/* 0 E> */ B(StackCheck),
/* 8 S> */ B(LdaConstant), U8(1),
B(Star), R(1),
@@ -101,15 +104,16 @@ snippet: "
function f() {}
f();
"
-frame size: 3
+frame size: 4
parameter count: 1
-bytecode array length: 26
+bytecode array length: 29
bytecodes: [
B(LdaConstant), U8(0),
B(Star), R(1),
B(LdaZero),
B(Star), R(2),
- B(CallRuntime), U16(Runtime::kDeclareGlobals), R(1), U8(2),
+ B(Mov), R(closure), R(3),
+ B(CallRuntime), U16(Runtime::kDeclareGlobalsForInterpreter), R(1), U8(3),
/* 0 E> */ B(StackCheck),
/* 16 S> */ B(LdrUndefined), R(2),
B(LdrGlobal), U8(1), R(1),
« no previous file with comments | « src/runtime/runtime-scopes.cc ('k') | test/cctest/interpreter/bytecode_expectations/TopLevelObjectLiterals.golden » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698