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

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

Issue 1985753002: [interpreter] Introduce fused bytecodes for common sequences. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebase. Created 4 years, 7 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/ForIn.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/ForIn.golden b/test/cctest/interpreter/bytecode_expectations/ForIn.golden
index cc2681fc17959ab6ef3ccbb6946199e66147fd15..1d0388d18784d21d33f722e295c15bb929d41364 100644
--- a/test/cctest/interpreter/bytecode_expectations/ForIn.golden
+++ b/test/cctest/interpreter/bytecode_expectations/ForIn.golden
@@ -154,24 +154,24 @@ snippet: "
"
frame size: 8
parameter count: 1
-bytecode array length: 94
+bytecode array length: 92
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
B(Star), R(1),
B(Star), R(0),
/* 77 S> */ B(CreateArrayLiteral), U8(1), U8(1), U8(3),
- B(JumpIfUndefined), U8(79),
- B(JumpIfNull), U8(77),
+ B(JumpIfUndefined), U8(77),
+ B(JumpIfNull), U8(75),
B(ToObject),
B(Star), R(1),
B(ForInPrepare), R(2),
B(LdaZero),
B(Star), R(5),
/* 68 S> */ B(ForInDone), R(5), R(4),
- B(JumpIfTrue), U8(64),
+ B(JumpIfTrue), U8(62),
B(ForInNext), R(1), R(5), R(2), U8(9),
- B(JumpIfUndefined), U8(51),
+ B(JumpIfUndefined), U8(49),
B(Star), R(6),
/* 67 E> */ B(Ldar), R(0),
B(Star), R(7),
@@ -180,23 +180,21 @@ bytecodes: [
/* 62 E> */ B(StackCheck),
/* 95 S> */ B(Ldar), R(0),
B(Star), R(6),
- /* 100 E> */ B(LoadIC), R(6), U8(2), U8(3),
- B(Star), R(7),
+ /* 100 E> */ B(LdrNamedProperty), R(6), U8(2), U8(3), R(7),
B(LdaSmi), U8(10),
/* 106 E> */ B(TestEqual), R(7),
B(JumpIfFalse), U8(4),
- /* 113 S> */ B(Jump), U8(20),
+ /* 113 S> */ B(Jump), U8(19),
/* 125 S> */ B(Ldar), R(0),
B(Star), R(6),
- /* 130 E> */ B(LoadIC), R(6), U8(2), U8(5),
- B(Star), R(7),
+ /* 130 E> */ B(LdrNamedProperty), R(6), U8(2), U8(5), R(7),
B(LdaSmi), U8(20),
/* 136 E> */ B(TestEqual), R(7),
B(JumpIfFalse), U8(4),
/* 143 S> */ B(Jump), U8(8),
B(ForInStep), R(5),
B(Star), R(5),
- B(Jump), U8(-65),
+ B(Jump), U8(-63),
B(LdaUndefined),
/* 152 S> */ B(Return),
]

Powered by Google App Engine
This is Rietveld 408576698