Skip to content

Commit 5c6633f

Browse files
authored
Merge pull request #10974 from ethereum/disable-reasoning-simplifier-fuzzing
Optimiser fuzzer: Disable reasoning based simplifier runs.
2 parents c812d92 + 91b7d60 commit 5c6633f

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

test/libyul/YulOptimizerTestCommon.cpp

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -390,10 +390,16 @@ string YulOptimizerTestCommon::randomOptimiserStep(unsigned _seed)
390390
if (count == idx)
391391
{
392392
string optimiserStep = step.first;
393-
// Do not fuzz mainFunction or wordSizeTransform
393+
// Do not fuzz mainFunction and wordSizeTransform
394394
// because they do not preserve yul code semantics.
395-
if (optimiserStep == "mainFunction" || optimiserStep == "wordSizeTransform")
396-
// "Fullsuite" is fuzzed roughly three times more frequently than
395+
// Do not fuzz reasoning based simplifier because
396+
// it can sometimes drain memory.
397+
if (
398+
optimiserStep == "mainFunction" ||
399+
optimiserStep == "wordSizeTransform" ||
400+
optimiserStep == "reasoningBasedSimplifier"
401+
)
402+
// "Fullsuite" is fuzzed roughly four times more frequently than
397403
// other steps because of the filtering in place above.
398404
return "fullSuite";
399405
else

0 commit comments

Comments
 (0)