Commit e398bf81 authored by Michael Starzinger's avatar Michael Starzinger Committed by Commit Bot

[test] Remove deprecated "fullcode" test variant.

Note that this also renames the existing "asm_wasm" variant to use the
more appropriate "stress_asm_wasm" name.

R=rmcilroy@chromium.org
BUG=v8:6409

Cq-Include-Trybots: master.tryserver.v8:v8_linux_noi18n_rel_ng
Change-Id: I1f9550cd03874c678f4583047a4e123a6f090250
Reviewed-on: https://chromium-review.googlesource.com/584879
Commit-Queue: Michael Starzinger <mstarzinger@chromium.org>
Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#46937}
parent 1d32273a
......@@ -86,24 +86,4 @@
'debug/debug-liveedit-restart-frame': [SKIP],
}], # 'arch == s390 or arch == s390x'
##############################################################################
['variant == fullcode', {
# fullcode doesn't generate precise return positions
'debug/debug-multiple-var-decl': [SKIP],
'debug/debug-step-into-valueof': [SKIP],
'regress/regress-5901-2': [SKIP],
'debug/debug-step-into-json': [SKIP],
'debug/es6/debug-step-destructuring-assignment': [SKIP],
'debug/es6/debug-step-into-regexp-subclass': [SKIP],
'debug/es8/async-debug-step-in': [SKIP],
'debug/debug-stepin-builtin-callback-opt': [SKIP],
'debug/debug-stepin-foreach': [SKIP],
'debug/es6/debug-stepin-default-parameters': [SKIP],
'debug/es6/debug-stepin-proxies': [SKIP],
'debug/es6/debug-promises/stepin-handler': [SKIP],
'debug/debug-liveedit-stepin': [SKIP],
'debug/es6/debug-stepin-microtasks': [SKIP],
'debug/ignition/elided-instruction': [SKIP],
'debug/es6/debug-stepin-string-template': [SKIP],
}], # variant == fullcode
]
......@@ -26,19 +26,4 @@
'*': [SKIP],
}], # variant == wasm_traps
##############################################################################
['variant == fullcode', {
# fullcode doesn't generate precise return positions
'debugger/return-break-locations': [SKIP],
'debugger/for-of-loops': [SKIP],
'debugger/break-locations-var-init': [SKIP],
'debugger/framework-precise-ranges': [SKIP],
'debugger/step-snapshot': [SKIP],
'debugger/step-into-next-script': [SKIP],
'debugger/asm-js-breakpoint-before-exec': [SKIP],
'debugger/break-locations-await': [SKIP],
'debugger/get-possible-breakpoints': [SKIP],
'debugger/stepping-with-blackboxed-ranges': [SKIP],
}], # variant == fullcode
]
......@@ -588,11 +588,6 @@
'regress/regress-353551': [SKIP],
}], # 'arch == ppc64'
##############################################################################
['variant == fullcode', {
'es6/array-iterator-turbo': [SKIP],
}], # variant == nocranshaft
##############################################################################
['variant == stress', {
'es6/array-iterator-turbo': [SKIP],
......
......@@ -726,11 +726,6 @@
'built-ins/Promise/prototype/then/deferred-is-resolved-value': [SKIP],
}], # asan == True
['asan and variant == fullcode', {
# Demands too many resources to test the old pipeline with asan.
'*': [SKIP],
}], # asan and variant == fullcode
['asan == True or msan == True or tsan == True', {
# https://bugs.chromium.org/p/v8/issues/detail?id=4639
# The failed allocation causes an asan/msan/tsan error
......
......@@ -112,10 +112,10 @@
}], # 'gcov_coverage'
##############################################################################
['variant == asm_wasm', {
['variant == stress_asm_wasm', {
# Stack size too large with asm.js validator.
'fast/js/excessive-comma-usage': [SKIP],
}], # variant == asm_wasm
}], # variant == stress_asm_wasm
##############################################################################
['variant == wasm_traps', {
......
......@@ -114,8 +114,7 @@ VARIANTS = ["default"]
MORE_VARIANTS = [
"stress",
"nooptimization",
"fullcode",
"asm_wasm",
"stress_asm_wasm",
"wasm_traps",
]
......
......@@ -6,12 +6,11 @@
ALL_VARIANT_FLAGS = {
"default": [[]],
"stress": [["--stress-opt", "--always-opt"]],
"fullcode": [["--noopt", "--stress-fullcodegen"]],
# No optimization means disable all optimizations. OptimizeFunctionOnNextCall
# would not force optimization too. It turns into a Nop. Please see
# https://chromium-review.googlesource.com/c/452620/ for more discussion.
"nooptimization": [["--noopt"]],
"asm_wasm": [["--validate-asm", "--stress-validate-asm", "--suppress-asm-messages"]],
"stress_asm_wasm": [["--validate-asm", "--stress-validate-asm", "--suppress-asm-messages"]],
"wasm_traps": [["--wasm_guard_pages", "--wasm_trap_handler", "--invoke-weak-callbacks"]],
}
......@@ -19,14 +18,13 @@ ALL_VARIANT_FLAGS = {
FAST_VARIANT_FLAGS = {
"default": [[]],
"stress": [["--stress-opt"]],
"fullcode": [["--noopt", "--stress-fullcodegen"]],
# No optimization means disable all optimizations. OptimizeFunctionOnNextCall
# would not force optimization too. It turns into a Nop. Please see
# https://chromium-review.googlesource.com/c/452620/ for more discussion.
"nooptimization": [["--noopt"]],
"asm_wasm": [["--validate-asm", "--stress-validate-asm", "--suppress-asm-messages"]],
"stress_asm_wasm": [["--validate-asm", "--stress-validate-asm", "--suppress-asm-messages"]],
"wasm_traps": [["--wasm_guard_pages", "--wasm_trap_handler", "--invoke-weak-callbacks"]],
}
ALL_VARIANTS = set(["default", "stress", "fullcode", "nooptimization",
"asm_wasm", "wasm_traps"])
ALL_VARIANTS = set(["default", "stress", "nooptimization", "stress_asm_wasm",
"wasm_traps"])
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