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

Unified Diff: test/mjsunit/shared-function-tier-up-default.js

Issue 2654733004: [tests] Make assertOptimized()/assertUnoptimized() great again. (Closed)
Patch Set: 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/mjsunit/shared-function-tier-up-default.js
diff --git a/test/mjsunit/shared-function-tier-up-default.js b/test/mjsunit/shared-function-tier-up-default.js
index 09c5f8e7fe88a5938419765c3bb6b48a8e8d3da3..7e8b7f45e79c0b5d1b8915974fad974f168db187 100644
--- a/test/mjsunit/shared-function-tier-up-default.js
+++ b/test/mjsunit/shared-function-tier-up-default.js
@@ -13,20 +13,24 @@
}
sum += f(i);
- if (%GetOptimizationStatus(f) == 3 || %GetOptimizationStatus(f) == 4) {
+ var opt_status = %GetOptimizationStatus(f);
+ if ((opt_status & V8OptimizationStatus.kAlwaysOptimize) ||
+ (opt_status & V8OptimizationStatus.kNeverOptimize)) {
// If we are always or never optimizing f, just exit, this test is useless.
return;
}
if (i == 1) {
// f must be baseline code.
- assertEquals(2, %GetOptimizationStatus(f));
+ assertFalse((opt_status & V8OptimizationStatus.kOptimized) !== 0);
+ assertFalse((opt_status & V8OptimizationStatus.kInterpreted) !== 0);
// Run twice (i = 0, 1), then tier-up.
%OptimizeFunctionOnNextCall(f);
} else if (i == 2) {
// Tier-up at i = 2 should go up to crankshaft.
- assertEquals(1, %GetOptimizationStatus(f));
+ assertTrue((opt_status & V8OptimizationStatus.kOptimized) !== 0);
+ assertFalse((opt_status & V8OptimizationStatus.kTurboFanned) !== 0);
}
}
})()

Powered by Google App Engine
This is Rietveld 408576698