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

Unified Diff: test/unittests/interpreter/bytecode-array-builder-unittest.cc

Issue 2331033002: [interpreter] Merge {OsrPoll} with {Jump} bytecode. (Closed)
Patch Set: Fix for wide jumps. Created 4 years, 3 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/unittests/interpreter/bytecode-array-builder-unittest.cc
diff --git a/test/unittests/interpreter/bytecode-array-builder-unittest.cc b/test/unittests/interpreter/bytecode-array-builder-unittest.cc
index d72743eb5f7d39a438e491e21574f65906473c94..86059051fd8745a6dee07223c29ad29d3d4f07ca 100644
--- a/test/unittests/interpreter/bytecode-array-builder-unittest.cc
+++ b/test/unittests/interpreter/bytecode-array-builder-unittest.cc
@@ -200,7 +200,8 @@ TEST_F(BytecodeArrayBuilderTest, AllBytecodesGenerated) {
.Bind(&after_jump)
.JumpIfNull(&start)
.JumpIfUndefined(&start)
- .JumpIfNotHole(&start);
+ .JumpIfNotHole(&start)
+ .JumpLoop(&start, 0);
}
// Longer jumps with constant operands
@@ -263,9 +264,6 @@ TEST_F(BytecodeArrayBuilderTest, AllBytecodesGenerated) {
// Emit stack check bytecode.
builder.StackCheck(0);
- // Emit an OSR poll bytecode.
- builder.OsrPoll(1);
-
// Emit throw and re-throw in it's own basic block so that the rest of the
// code isn't omitted due to being dead.
BytecodeLabel after_throw;

Powered by Google App Engine
This is Rietveld 408576698