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

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

Issue 2187823002: [interpreter] Don't keep postfix result if only visited for effect (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: regen all golden files Created 4 years, 5 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/JumpsRequiringConstantWideOperands.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden b/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden
index f7c17db4fbac1a45028fbc9beb61c2091f279370..d22e5ec8360f099af7e34a9655d731ea85c8d5f4 100644
--- a/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden
+++ b/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden
@@ -327,9 +327,9 @@ snippet: "
}
return 3;
"
-frame size: 3
+frame size: 2
parameter count: 1
-bytecode array length: 1409
+bytecode array length: 1407
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaConstant), U8(0),
@@ -960,7 +960,7 @@ bytecodes: [
B(Star), R(1),
/* 4108 S> */ B(LdaSmi), U8(3),
/* 4108 E> */ B(TestLessThan), R(1),
- B(Wide), B(JumpIfFalse), U16(37),
+ B(Wide), B(JumpIfFalse), U16(35),
/* 4090 E> */ B(StackCheck),
/* 4122 S> */ B(LdaSmi), U8(1),
/* 4128 E> */ B(TestEqual), R(1),
@@ -969,12 +969,11 @@ bytecodes: [
/* 4146 S> */ B(LdaSmi), U8(2),
/* 4152 E> */ B(TestEqual), R(1),
B(Wide), B(JumpIfFalse), U16(7),
- /* 4158 S> */ B(Wide), B(Jump), U16(12),
+ /* 4158 S> */ B(Wide), B(Jump), U16(10),
/* 4114 S> */ B(Ldar), R(1),
- B(ToNumber), R(2),
B(Inc),
B(Star), R(1),
- B(Jump), U8(-40),
+ B(Jump), U8(-38),
/* 4167 S> */ B(LdaSmi), U8(3),
/* 4177 S> */ B(Return),
]

Powered by Google App Engine
This is Rietveld 408576698