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

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

Issue 2640273002: [ignition] Expect 'I' for signed bytecode operands (Closed)
Patch Set: Rebase Created 3 years, 11 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 8cea991aebdaf40cf4071407de075e8e1f3a1fe9..292247b4258134c67cdd97ec9bda0a29433de42d 100644
--- a/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden
+++ b/test/cctest/interpreter/bytecode_expectations/JumpsRequiringConstantWideOperands.golden
@@ -956,23 +956,23 @@ bytecodes: [
B(Star), R(0),
/* 4103 S> */ B(LdaZero),
B(Star), R(1),
- /* 4108 S> */ B(LdaSmi), U8(3),
+ /* 4108 S> */ B(LdaSmi), I8(3),
/* 4108 E> */ B(TestLessThan), R(1), U8(2),
B(Wide), B(JumpIfFalse), U16(39),
/* 4090 E> */ B(StackCheck),
- /* 4122 S> */ B(LdaSmi), U8(1),
+ /* 4122 S> */ B(LdaSmi), I8(1),
/* 4128 E> */ B(TestEqual), R(1), U8(4),
B(Wide), B(JumpIfFalse), U16(7),
/* 4134 S> */ B(Wide), B(Jump), U16(16),
- /* 4146 S> */ B(LdaSmi), U8(2),
+ /* 4146 S> */ B(LdaSmi), I8(2),
/* 4152 E> */ B(TestEqual), R(1), U8(5),
B(Wide), B(JumpIfFalse), U16(7),
/* 4158 S> */ B(Wide), B(Jump), U16(12),
/* 4114 S> */ B(Ldar), R(1),
B(Inc), U8(3),
B(Star), R(1),
- B(JumpLoop), U8(42), U8(0),
- /* 4167 S> */ B(LdaSmi), U8(3),
+ B(JumpLoop), U8(42), I8(0),
+ /* 4167 S> */ B(LdaSmi), I8(3),
/* 4177 S> */ B(Return),
]
constant pool: [

Powered by Google App Engine
This is Rietveld 408576698