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

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

Issue 2485383002: [Interpreter] Remove Ldr[Named/Keyed]Property bytecodes and use Star Lookahead instead. (Closed)
Patch Set: Created 4 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 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 a23bb9022643bb43f602deca9cadc0c91d99bde7..c8cbc7a61dfc46f174354e1fd8ce5c930e73a169 100644
--- a/test/cctest/interpreter/bytecode_expectations/ForIn.golden
+++ b/test/cctest/interpreter/bytecode_expectations/ForIn.golden
@@ -147,40 +147,42 @@ snippet: "
"
frame size: 7
parameter count: 1
-bytecode array length: 83
+bytecode array length: 85
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1), R(1),
B(Mov), R(1), R(0),
/* 77 S> */ B(CreateArrayLiteral), U8(1), U8(1), U8(9),
- B(JumpIfUndefined), U8(68),
- B(JumpIfNull), U8(66),
+ B(JumpIfUndefined), U8(70),
+ B(JumpIfNull), U8(68),
B(ToObject), R(1),
B(ForInPrepare), R(1), R(2),
B(LdaZero),
B(Star), R(5),
/* 68 S> */ B(ForInContinue), R(5), R(4),
- B(JumpIfFalse), U8(53),
+ B(JumpIfFalse), U8(55),
B(ForInNext), R(1), R(5), R(2), U8(12),
- B(JumpIfUndefined), U8(39),
+ B(JumpIfUndefined), U8(41),
B(Star), R(6),
/* 67 E> */ B(StaNamedPropertySloppy), R(0), U8(2), U8(10),
/* 62 E> */ B(StackCheck),
/* 95 S> */ B(Nop),
- /* 100 E> */ B(LdrNamedProperty), R(0), U8(2), U8(4), R(6),
+ /* 100 E> */ B(LdaNamedProperty), R(0), U8(2), U8(4),
+ B(Star), R(6),
B(LdaSmi), U8(10),
/* 106 E> */ B(TestEqual), R(6), U8(6),
B(JumpIfFalse), U8(4),
- /* 113 S> */ B(Jump), U8(17),
+ /* 113 S> */ B(Jump), U8(18),
/* 125 S> */ B(Nop),
- /* 130 E> */ B(LdrNamedProperty), R(0), U8(2), U8(7), R(6),
+ /* 130 E> */ B(LdaNamedProperty), R(0), U8(2), U8(7),
+ B(Star), R(6),
B(LdaSmi), U8(20),
/* 136 E> */ B(TestEqual), R(6), U8(9),
B(JumpIfFalse), U8(4),
/* 143 S> */ B(Jump), U8(9),
B(ForInStep), R(5),
B(Star), R(5),
- B(JumpLoop), U8(-53), U8(0),
+ B(JumpLoop), U8(-55), U8(0),
B(LdaUndefined),
/* 152 S> */ B(Return),
]

Powered by Google App Engine
This is Rietveld 408576698