From 65f8948a08cf4c1a300116840cdeca7773eb62c0 Mon Sep 17 00:00:00 2001 From: Michael Achenbach Date: Wed, 8 Sep 2021 12:45:22 +0200 Subject: [PATCH] [test] Neuter stack-size for more tests incompatible with stack-size fuzzing Also bump the minimum size determined by manually testing with: mjsunit/harmony/modules-import-13 Bug: v8:12197 Change-Id: Ic21d42547ba14cd927d158388ad4bc6f1891f45c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3147453 Commit-Queue: Michael Achenbach Commit-Queue: Leszek Swirski Auto-Submit: Michael Achenbach Reviewed-by: Leszek Swirski Cr-Commit-Position: refs/heads/main@{#76723} --- test/mjsunit/compiler/regress-9017.js | 2 ++ tools/testrunner/testproc/fuzzer.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/test/mjsunit/compiler/regress-9017.js b/test/mjsunit/compiler/regress-9017.js index c484e177c6..72652f22f4 100644 --- a/test/mjsunit/compiler/regress-9017.js +++ b/test/mjsunit/compiler/regress-9017.js @@ -5,6 +5,8 @@ // Flags: --allow-natives-syntax --noturbo-inlining --noturbo-verify-allocation // This test invokes optimization manually, no need for stress modes: // Flags: --nostress-opt --noalways-opt +// This neuters too low stack size passed by the flag fuzzer. +// Flags: --stack-size=864 // Ensure that very large stack frames can be used successfully. // The flag --noturbo-verify-allocation is to make this run a little faster; it diff --git a/tools/testrunner/testproc/fuzzer.py b/tools/testrunner/testproc/fuzzer.py index 0452704781..67250b1c74 100644 --- a/tools/testrunner/testproc/fuzzer.py +++ b/tools/testrunner/testproc/fuzzer.py @@ -269,7 +269,7 @@ class CompactionFuzzer(Fuzzer): class StackSizeFuzzer(Fuzzer): def create_flags_generator(self, rng, test, analysis_value): while True: - yield ['--stack-size=%d' % rng.randint(50, 983)] + yield ['--stack-size=%d' % rng.randint(54, 983)] class TaskDelayFuzzer(Fuzzer): def create_flags_generator(self, rng, test, analysis_value):