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

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

Issue 1997653002: [interpreter] Bytecode register optimizer. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Try harder with source positions. 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/LogicalExpressions.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/LogicalExpressions.golden b/test/cctest/interpreter/bytecode_expectations/LogicalExpressions.golden
index fed8367212b4e9a1dc3402167e164c163a761f14..70aba1db81550c10c776b9167398a7f7f2580dfb 100644
--- a/test/cctest/interpreter/bytecode_expectations/LogicalExpressions.golden
+++ b/test/cctest/interpreter/bytecode_expectations/LogicalExpressions.golden
@@ -31,16 +31,15 @@ handlers: [
snippet: "
var x = 0; return (x == 1) || 3;
"
-frame size: 2
+frame size: 1
parameter count: 1
-bytecode array length: 15
+bytecode array length: 13
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaZero),
/* 42 E> */ B(Star), R(0),
- /* 45 S> */ B(Star), R(1),
- B(LdaSmi), U8(1),
- /* 55 E> */ B(TestEqual), R(1),
+ /* 45 S> */ B(LdaSmi), U8(1),
+ /* 55 E> */ B(TestEqual), R(0),
B(JumpIfTrue), U8(4),
B(LdaSmi), U8(3),
/* 67 S> */ B(Return),
@@ -74,16 +73,15 @@ handlers: [
snippet: "
var x = 0; return (x == 0) && 3;
"
-frame size: 2
+frame size: 1
parameter count: 1
-bytecode array length: 14
+bytecode array length: 12
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaZero),
/* 42 E> */ B(Star), R(0),
- /* 45 S> */ B(Star), R(1),
- B(LdaZero),
- /* 55 E> */ B(TestEqual), R(1),
+ /* 45 S> */ B(LdaZero),
+ /* 55 E> */ B(TestEqual), R(0),
B(JumpIfFalse), U8(4),
B(LdaSmi), U8(3),
/* 67 S> */ B(Return),
@@ -548,9 +546,9 @@ snippet: "
a = 1, b = 2,
a = 1, b = 2, 3);
"
-frame size: 4
+frame size: 3
parameter count: 1
-bytecode array length: 282
+bytecode array length: 278
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaSmi), U8(1),
@@ -559,10 +557,8 @@ bytecodes: [
/* 53 E> */ B(Star), R(1),
/* 60 S> */ B(LdaSmi), U8(3),
/* 60 E> */ B(Star), R(2),
- /* 63 S> */ B(Ldar), R(0),
- B(Star), R(3),
- B(LdaSmi), U8(3),
- /* 73 E> */ B(TestGreaterThan), R(3),
+ /* 63 S> */ B(LdaSmi), U8(3),
+ /* 73 E> */ B(TestGreaterThan), R(0),
B(JumpIfTrueConstant), U8(0),
B(LdaSmi), U8(1),
/* 87 E> */ B(Star), R(1),
@@ -737,9 +733,9 @@ snippet: "
a = 1, b = 2,
a = 1, b = 2, 3);
"
-frame size: 4
+frame size: 3
parameter count: 1
-bytecode array length: 281
+bytecode array length: 277
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaZero),
@@ -748,10 +744,8 @@ bytecodes: [
/* 53 E> */ B(Star), R(1),
/* 60 S> */ B(LdaSmi), U8(3),
/* 60 E> */ B(Star), R(2),
- /* 63 S> */ B(Ldar), R(0),
- B(Star), R(3),
- B(LdaSmi), U8(5),
- /* 73 E> */ B(TestLessThan), R(3),
+ /* 63 S> */ B(LdaSmi), U8(5),
+ /* 73 E> */ B(TestLessThan), R(0),
B(JumpIfFalseConstant), U8(0),
B(LdaSmi), U8(1),
/* 87 E> */ B(Star), R(1),

Powered by Google App Engine
This is Rietveld 408576698