[fuzzing] Re-enable fuzzing for staged harmony features
The fuzzers were passing the flag --es-staging which doesn't exist. This CL updates them to pass the flag --harmony which does exist. Change-Id: I02c83026e5b9bdf49e51e700f16702bf56cd49e8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3522064 Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#79464}
This commit is contained in:
parent
7e7661a1c8
commit
3a43f6552e
@ -316,7 +316,7 @@ DEFINE_BOOL(harmony_shipping, true, "enable all shipped harmony features")
|
||||
#define HARMONY_INPROGRESS(V) HARMONY_INPROGRESS_BASE(V)
|
||||
#endif
|
||||
|
||||
// Features that are complete (but still behind --harmony/es-staging flag).
|
||||
// Features that are complete (but still behind the --harmony flag).
|
||||
#define HARMONY_STAGED_BASE(V) \
|
||||
V(harmony_array_grouping, "harmony array grouping")
|
||||
|
||||
|
@ -206,18 +206,18 @@ class CombinedTest(testcase.D8TestCase):
|
||||
"""In addition to standard set of shell flags it appends:
|
||||
--disable-abortjs: %AbortJS can abort the test even inside
|
||||
trycatch-wrapper, so we disable it.
|
||||
--es-staging: We skip all harmony flags due to false positives,
|
||||
--harmony: We skip all harmony flags due to false positives,
|
||||
but always pass the staging flag to cover the mature features.
|
||||
--omit-quit: Calling quit() in JS would otherwise early terminate.
|
||||
--quiet-load: suppress any stdout from load() function used by
|
||||
trycatch-wrapper.
|
||||
"""
|
||||
return [
|
||||
'--test',
|
||||
'--disable-abortjs',
|
||||
'--es-staging',
|
||||
'--omit-quit',
|
||||
'--quiet-load',
|
||||
'--test',
|
||||
'--disable-abortjs',
|
||||
'--harmony',
|
||||
'--omit-quit',
|
||||
'--quiet-load',
|
||||
]
|
||||
|
||||
def _get_cmd_params(self):
|
||||
|
@ -9,9 +9,9 @@
|
||||
# Compared x64,ignition with x64,ignition_turbo
|
||||
#
|
||||
# Flags of x64,ignition:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up --flag1 --flag2=0
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up --flag1 --flag2=0
|
||||
# Flags of x64,ignition_turbo:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --flag3
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --flag3
|
||||
#
|
||||
# Difference:
|
||||
- unknown
|
||||
|
@ -9,9 +9,9 @@
|
||||
# Compared x64,ignition with x64,ignition_turbo
|
||||
#
|
||||
# Flags of x64,ignition:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
# Flags of x64,ignition_turbo:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --bad-flag
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --bad-flag
|
||||
#
|
||||
# Difference:
|
||||
+ bad behavior
|
||||
|
@ -9,9 +9,9 @@
|
||||
# Compared x64,ignition with ia32,ignition_turbo
|
||||
#
|
||||
# Flags of x64,ignition:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
# Flags of ia32,ignition_turbo:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --very-bad-flag
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --very-bad-flag
|
||||
#
|
||||
# Difference:
|
||||
+ very bad behavior
|
||||
|
@ -9,9 +9,9 @@
|
||||
# Compared x64,ignition with x64,ignition_turbo
|
||||
#
|
||||
# Flags of x64,ignition:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345 --turbo-filter=~ --no-opt --no-sparkplug --liftoff --no-wasm-tier-up
|
||||
# Flags of x64,ignition_turbo:
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --es-staging --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345
|
||||
--correctness-fuzzer-suppressions --expose-gc --fuzzing --allow-natives-for-differential-fuzzing --invoke-weak-callbacks --omit-quit --harmony --wasm-staging --no-wasm-async-compilation --suppress-asm-messages --random-seed 12345
|
||||
#
|
||||
# Difference:
|
||||
- unknown
|
||||
|
@ -16,16 +16,16 @@ PYTHON3 = sys.version_info >= (3, 0)
|
||||
|
||||
# List of default flags passed to each d8 run.
|
||||
DEFAULT_FLAGS = [
|
||||
'--correctness-fuzzer-suppressions',
|
||||
'--expose-gc',
|
||||
'--fuzzing',
|
||||
'--allow-natives-for-differential-fuzzing',
|
||||
'--invoke-weak-callbacks',
|
||||
'--omit-quit',
|
||||
'--es-staging',
|
||||
'--wasm-staging',
|
||||
'--no-wasm-async-compilation',
|
||||
'--suppress-asm-messages',
|
||||
'--correctness-fuzzer-suppressions',
|
||||
'--expose-gc',
|
||||
'--fuzzing',
|
||||
'--allow-natives-for-differential-fuzzing',
|
||||
'--invoke-weak-callbacks',
|
||||
'--omit-quit',
|
||||
'--harmony',
|
||||
'--wasm-staging',
|
||||
'--no-wasm-async-compilation',
|
||||
'--suppress-asm-messages',
|
||||
]
|
||||
|
||||
BASE_PATH = os.path.dirname(os.path.abspath(__file__))
|
||||
|
@ -97,14 +97,14 @@ const DISALLOWED_FLAGS = [
|
||||
// stabilized yet and would cause too much noise when enabled.
|
||||
/^--experimental-.*/,
|
||||
|
||||
// Disallowed due to noise. We explicitly add --es-staging to job
|
||||
// Disallowed due to noise. We explicitly add --harmony to job
|
||||
// definitions, and all of these features are staged before launch.
|
||||
/^--harmony-.*/,
|
||||
|
||||
// Disallowed because they are passed explicitly on the command line.
|
||||
'--allow-natives-syntax',
|
||||
'--debug-code',
|
||||
'--es-staging',
|
||||
'--harmony',
|
||||
'--wasm-staging',
|
||||
'--expose-gc',
|
||||
'--expose_gc',
|
||||
|
Loading…
Reference in New Issue
Block a user