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

Unified Diff: test/mjsunit/mjsunit.js

Issue 2655263004: [tests] Fix assertUnoptimized() when --deopt-every-n-times flag is used. (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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/mjsunit.js
diff --git a/test/mjsunit/mjsunit.js b/test/mjsunit/mjsunit.js
index ad8c511bc85fdcd6770360b886e7dfd2c7e1e243..ec17f1d76afe8be552042bdbc27a233d846c3f9f 100644
--- a/test/mjsunit/mjsunit.js
+++ b/test/mjsunit/mjsunit.js
@@ -500,6 +500,12 @@ var isTurboFanned;
assertFalse((opt_status & V8OptimizationStatus.kAlwaysOptimize) !== 0,
"test does not make sense with --always-opt");
assertTrue((opt_status & V8OptimizationStatus.kIsFunction) !== 0, name_opt);
+ if ((opt_status & V8OptimizationStatus.kMaybeDeopted) !== 0) {
+ // When --deopt-every-n-times flag is specified it's no longer guaranteed
+ // that particular function is still deoptimized, so keep running the test
+ // to stress test the deoptimizer.
+ return;
+ }
assertFalse((opt_status & V8OptimizationStatus.kOptimized) !== 0, name_opt);
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698