Commit 28c25b5f authored by Michal Majewski's avatar Michal Majewski Committed by Commit Bot

Fixed/skipped tests for deoptimization fuzzing.

Skipped the tests that are not suitable for deoptimization fuzzing.
regress/regress-2618 test fixed to check kMaybeDeopted flag.
Minor code style fix in mjsunit.js.

Bug: v8:6900
Change-Id: Icc02a6b99005ae08ee7cb6cf2c1e9137329d79d3
Reviewed-on: https://chromium-review.googlesource.com/708797
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#48444}
parent 9dc6dd86
......@@ -582,8 +582,8 @@ var failWithMessage;
return OptimizationStatusImpl(fun, sync_opt);
}
assertUnoptimized = function assertUnoptimized(fun, sync_opt, name_opt,
skip_if_maybe_deopted = true) {
assertUnoptimized = function assertUnoptimized(
fun, sync_opt, name_opt, skip_if_maybe_deopted = true) {
if (sync_opt === undefined) sync_opt = "";
var opt_status = OptimizationStatus(fun, sync_opt);
// Tests that use assertUnoptimized() do not make sense if --always-opt
......@@ -601,8 +601,8 @@ var failWithMessage;
assertFalse((opt_status & V8OptimizationStatus.kOptimized) !== 0, name_opt);
}
assertOptimized = function assertOptimized(fun, sync_opt, name_opt,
skip_if_maybe_deopted = true) {
assertOptimized = function assertOptimized(
fun, sync_opt, name_opt, skip_if_maybe_deopted = true) {
if (sync_opt === undefined) sync_opt = "";
var opt_status = OptimizationStatus(fun, sync_opt);
// Tests that use assertOptimized() do not make sense if --no-opt
......
......@@ -559,6 +559,10 @@
'regress/regress-2185-2': [SKIP],
'readonly': [SKIP],
'array-feedback': [SKIP],
'deopt-recursive-eager-once': [SKIP],
'deopt-recursive-lazy-once': [SKIP],
'deopt-recursive-soft-once': [SKIP],
'code-coverage-block-opt': [SKIP],
# Bounds check triggers forced deopt for array constructors.
'array-constructor-feedback': [SKIP],
......
......@@ -40,6 +40,7 @@ function f() {
// feedback.
var opt_status = %GetOptimizationStatus(f);
assertTrue(
(opt_status & V8OptimizationStatus.kMaybeDeopted) !== 0 ||
(opt_status & V8OptimizationStatus.kTopmostFrameIsTurboFanned) !== 0);
} while (false);
} while (false);
......@@ -66,8 +67,10 @@ function g() {
do {
for (var i = 0; i < 10; i++) %OptimizeOsr();
var opt_status = %GetOptimizationStatus(g);
assertTrue((opt_status
& V8OptimizationStatus.kTopmostFrameIsTurboFanned) !== 0);
assertTrue(
(opt_status & V8OptimizationStatus.kMaybeDeopted) !== 0 ||
(opt_status &
V8OptimizationStatus.kTopmostFrameIsTurboFanned) !== 0);
} while (false);
} while (false);
} while (false);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment