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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 1997653002: [interpreter] Bytecode register optimizer. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Decouple a test from implementation. 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
« no previous file with comments | « test/cctest/interpreter/bytecode_expectations/UnaryOperators.golden ('k') | test/cctest/interpreter/bytecode_expectations/WithStatement.golden » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/interpreter/bytecode_expectations/WideRegisters.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/WideRegisters.golden b/test/cctest/interpreter/bytecode_expectations/WideRegisters.golden
index 2a617b97091d4bd74949e231222e820bcc9136e0..36120798d30ad8063fba4b8dfaa94f88bc4b68c0 100644
--- a/test/cctest/interpreter/bytecode_expectations/WideRegisters.golden
+++ b/test/cctest/interpreter/bytecode_expectations/WideRegisters.golden
@@ -171,12 +171,12 @@ snippet: "
"
frame size: 157
parameter count: 1
-bytecode array length: 9
+bytecode array length: 11
bytecodes: [
/* 30 E> */ B(StackCheck),
- /* 1494 S> */ B(Wide), B(Ldar), R16(127),
- B(Star), R(0),
- /* 1505 S> */ B(Nop),
+ /* 1494 S> */ B(Nop),
+ B(Wide), B(Mov), R16(127), R16(0),
+ /* 1505 S> */ B(Ldar), R(0),
/* 1516 S> */ B(Return),
]
constant pool: [
@@ -348,12 +348,12 @@ snippet: "
"
frame size: 157
parameter count: 1
-bytecode array length: 11
+bytecode array length: 13
bytecodes: [
/* 30 E> */ B(StackCheck),
- /* 1494 S> */ B(Wide), B(Ldar), R16(126),
- B(Wide), B(Star), R16(127),
- /* 1507 S> */ B(Nop),
+ /* 1494 S> */ B(Nop),
+ B(Wide), B(Mov), R16(126), R16(127),
+ /* 1507 S> */ B(Wide), B(Ldar), R16(127),
/* 1520 S> */ B(Return),
]
constant pool: [
@@ -523,15 +523,13 @@ snippet: "
if (x2 > 3) { return x129; }
return x128;
"
-frame size: 158
+frame size: 157
parameter count: 1
-bytecode array length: 25
+bytecode array length: 19
bytecodes: [
/* 30 E> */ B(StackCheck),
- /* 1494 S> */ B(Ldar), R(2),
- B(Wide), B(Star), R16(157),
- B(LdaSmi), U8(3),
- /* 1501 E> */ B(Wide), B(TestGreaterThan), R16(157),
+ /* 1494 S> */ B(LdaSmi), U8(3),
+ /* 1501 E> */ B(Wide), B(TestGreaterThan), R16(2),
B(JumpIfFalse), U8(7),
/* 1508 S> */ B(Wide), B(Ldar), R16(129),
/* 1536 S> */ B(Return),
@@ -707,24 +705,21 @@ snippet: "
if (x2 > 3) { return x0; }
return x129;
"
-frame size: 158
+frame size: 157
parameter count: 1
-bytecode array length: 48
+bytecode array length: 39
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 1503 S> */ B(LdaZero),
B(Star), R(0),
- /* 1506 S> */ B(Wide), B(Ldar), R16(129),
- B(Wide), B(Star), R16(157),
- B(LdaSmi), U8(3),
- /* 1515 E> */ B(Wide), B(TestEqual), R16(157),
- B(JumpIfFalse), U8(8),
- /* 1534 S> */ B(Ldar), R(0),
- /* 1534 E> */ B(Wide), B(Star), R16(129),
- /* 1540 S> */ B(Ldar), R(2),
- B(Wide), B(Star), R16(157),
- B(LdaSmi), U8(3),
- /* 1547 E> */ B(Wide), B(TestGreaterThan), R16(157),
+ /* 1506 S> */ B(LdaSmi), U8(3),
+ /* 1515 E> */ B(Wide), B(TestEqual), R16(129),
+ B(JumpIfFalse), U8(13),
+ /* 1534 S> */ B(Nop),
+ /* 1534 E> */ B(Wide), B(Mov), R16(0), R16(129),
+ B(Wide), B(Ldar), R16(129),
+ /* 1540 S> */ B(LdaSmi), U8(3),
+ /* 1547 E> */ B(Wide), B(TestGreaterThan), R16(2),
B(JumpIfFalse), U8(5),
/* 1554 S> */ B(Ldar), R(0),
/* 1580 S> */ B(Return),
@@ -901,7 +896,7 @@ snippet: "
"
frame size: 158
parameter count: 1
-bytecode array length: 66
+bytecode array length: 59
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 1503 S> */ B(LdaZero),
@@ -910,23 +905,21 @@ bytecodes: [
B(Star), R(1),
/* 1523 S> */ B(LdaZero),
/* 1528 E> */ B(Wide), B(Star), R16(128),
- /* 1538 S> */ B(Wide), B(Ldar), R16(128),
- B(Wide), B(Star), R16(157),
- B(LdaSmi), U8(64),
- /* 1538 E> */ B(Wide), B(TestLessThan), R16(157),
- B(JumpIfFalse), U8(35),
+ /* 1538 S> */ B(LdaSmi), U8(64),
+ /* 1538 E> */ B(Wide), B(TestLessThan), R16(128),
+ B(JumpIfFalse), U8(36),
/* 1518 E> */ B(StackCheck),
- /* 1555 S> */ B(Ldar), R(1),
- B(Wide), B(Star), R16(157),
+ /* 1555 S> */ B(Nop),
/* 1561 E> */ B(Wide), B(Ldar), R16(128),
- B(Wide), B(Add), R16(157),
+ B(Wide), B(Add), R16(1),
+ B(Wide), B(Mov), R16(1), R16(157),
/* 1558 E> */ B(Star), R(1),
/* 1548 S> */ B(Wide), B(Ldar), R16(128),
B(ToNumber),
B(Wide), B(Star), R16(157),
B(Inc),
/* 1548 E> */ B(Wide), B(Star), R16(128),
- B(Jump), U8(-47),
+ B(Jump), U8(-40),
/* 1567 S> */ B(Wide), B(Ldar), R16(128),
/* 1580 S> */ B(Return),
]
@@ -1100,7 +1093,7 @@ snippet: "
"
frame size: 163
parameter count: 1
-bytecode array length: 84
+bytecode array length: 85
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 1503 S> */ B(Wide), B(LdaSmi), U16(1234),
@@ -1108,27 +1101,27 @@ bytecodes: [
/* 1518 S> */ B(LdaZero),
B(Star), R(1),
/* 1534 S> */ B(Ldar), R(0),
- B(JumpIfUndefined), U8(69),
- B(JumpIfNull), U8(67),
+ B(JumpIfUndefined), U8(70),
+ B(JumpIfNull), U8(68),
B(ToObject),
- B(Wide), B(Star), R16(157),
B(Wide), B(ForInPrepare), R16(158),
+ B(Wide), B(Star), R16(157),
B(LdaZero),
B(Wide), B(Star), R16(161),
/* 1526 S> */ B(Wide), B(ForInDone), R16(161), R16(160),
- B(JumpIfTrue), U8(45),
+ B(JumpIfTrue), U8(46),
B(Wide), B(ForInNext), R16(157), R16(161), R16(158), U16(1),
- B(JumpIfUndefined), U8(23),
+ B(JumpIfUndefined), U8(24),
B(Wide), B(Star), R16(128),
/* 1521 E> */ B(StackCheck),
- /* 1541 S> */ B(Ldar), R(1),
- B(Wide), B(Star), R16(162),
+ /* 1541 S> */ B(Nop),
/* 1547 E> */ B(Wide), B(Ldar), R16(128),
- B(Wide), B(Add), R16(162),
+ B(Wide), B(Add), R16(1),
+ B(Wide), B(Mov), R16(1), R16(162),
/* 1544 E> */ B(Star), R(1),
B(Wide), B(ForInStep), R16(161),
B(Wide), B(Star), R16(161),
- B(Jump), U8(-49),
+ B(Jump), U8(-50),
/* 1553 S> */ B(Ldar), R(1),
/* 1564 S> */ B(Return),
]
@@ -1306,16 +1299,14 @@ parameter count: 1
bytecode array length: 55
bytecodes: [
/* 30 E> */ B(StackCheck),
- /* 1494 S> */ B(Ldar), R(64),
- B(Wide), B(Star), R16(157),
- /* 1509 E> */ B(Ldar), R(63),
- B(Wide), B(Star), R16(158),
+ /* 1494 S> */ B(Nop),
+ /* 1509 E> */ B(Wide), B(Mov), R16(64), R16(157),
+ B(Wide), B(Mov), R16(63), R16(158),
B(Wide), B(CallRuntime), U16(Runtime::kAdd), R16(157), U16(2),
B(Star), R(0),
- /* 1515 S> */ B(Ldar), R(27),
- B(Wide), B(Star), R16(157),
- /* 1530 E> */ B(Wide), B(Ldar), R16(143),
- B(Wide), B(Star), R16(158),
+ /* 1515 S> */ B(Nop),
+ /* 1530 E> */ B(Wide), B(Mov), R16(27), R16(157),
+ B(Wide), B(Mov), R16(143), R16(158),
B(Wide), B(CallRuntime), U16(Runtime::kAdd), R16(157), U16(2),
B(Star), R(1),
/* 1537 S> */ B(CallRuntime), U16(Runtime::kTheHole), R(0), U8(0),
« no previous file with comments | « test/cctest/interpreter/bytecode_expectations/UnaryOperators.golden ('k') | test/cctest/interpreter/bytecode_expectations/WithStatement.golden » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698