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

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

Issue 2286273002: [interpreter] Make the comparison bytecode handlers collect type feedback. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix. Created 4 years, 4 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 aaf8d110d21ba4fdaed91f7691343c2307961ac4..88b8d7a082419e229b638f37706349f8357cee75 100644
--- a/test/cctest/interpreter/bytecode_expectations/ForIn.golden
+++ b/test/cctest/interpreter/bytecode_expectations/ForIn.golden
@@ -149,40 +149,40 @@ snippet: "
"
frame size: 8
parameter count: 1
-bytecode array length: 80
+bytecode array length: 82
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(3),
- B(JumpIfUndefined), U8(65),
- B(JumpIfNull), U8(63),
+ B(JumpIfUndefined), U8(67),
+ B(JumpIfNull), U8(65),
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(50),
- B(ForInNext), R(1), R(5), R(2), U8(9),
- B(JumpIfUndefined), U8(37),
+ B(JumpIfFalse), U8(52),
+ B(ForInNext), R(1), R(5), R(2), U8(11),
+ B(JumpIfUndefined), U8(39),
B(Star), R(6),
- /* 67 E> */ B(StaNamedPropertySloppy), R(0), U8(2), U8(7),
+ /* 67 E> */ B(StaNamedPropertySloppy), R(0), U8(2), U8(9),
/* 62 E> */ B(StackCheck),
/* 95 S> */ B(Nop),
/* 100 E> */ B(LdrNamedProperty), R(0), U8(2), U8(3), R(7),
B(LdaSmi), U8(10),
- /* 106 E> */ B(TestEqual), R(7),
+ /* 106 E> */ B(TestEqual), R(7), U8(5),
B(JumpIfFalse), U8(4),
- /* 113 S> */ B(Jump), U8(16),
+ /* 113 S> */ B(Jump), U8(17),
/* 125 S> */ B(Nop),
- /* 130 E> */ B(LdrNamedProperty), R(0), U8(2), U8(5), R(7),
+ /* 130 E> */ B(LdrNamedProperty), R(0), U8(2), U8(6), R(7),
B(LdaSmi), U8(20),
- /* 136 E> */ B(TestEqual), R(7),
+ /* 136 E> */ B(TestEqual), R(7), U8(8),
B(JumpIfFalse), U8(4),
/* 143 S> */ B(Jump), U8(8),
B(ForInStep), R(5),
B(Star), R(5),
- B(Jump), U8(-51),
+ B(Jump), U8(-53),
B(LdaUndefined),
/* 152 S> */ B(Return),
]

Powered by Google App Engine
This is Rietveld 408576698