Skip to content

Optimize fuzzing using targeted substitution#77

Merged
automergerpr-permission-manager[bot] merged 10 commits intomasterfrom
optimize-subst
Feb 13, 2026
Merged

Optimize fuzzing using targeted substitution#77
automergerpr-permission-manager[bot] merged 10 commits intomasterfrom
optimize-subst

Conversation

@bbyalcinkaya
Copy link
Member

This PR improves performance by optimizing the substitution algorithm in the fuzzer. Substitutions now target only the <program> cell via the subst_func parameter in fuzz. This prevents the substitution from needlessly traversing these large terms, leading to a 3x speedup in fuzzing.

The speedup was measured using the existing test suite, particularly the FxDAO tests. In cases where the initial state is larger—such as tests involving multiple contracts or bigger contracts—the performance improvement is even more significant.

@bbyalcinkaya bbyalcinkaya marked this pull request as ready for review May 5, 2025 09:48
@bbyalcinkaya bbyalcinkaya requested a review from gtrepta May 5, 2025 09:49
@bbyalcinkaya bbyalcinkaya force-pushed the optimize-subst branch 2 times, most recently from 9ef8fa8 to a3b6f93 Compare October 31, 2025 11:22
@bbyalcinkaya bbyalcinkaya force-pushed the optimize-subst branch 3 times, most recently from 07b3f53 to a39ef1c Compare December 10, 2025 12:25
@bbyalcinkaya bbyalcinkaya force-pushed the optimize-subst branch 2 times, most recently from 1cf3bf0 to e79e113 Compare January 28, 2026 06:16
@automergerpr-permission-manager automergerpr-permission-manager bot merged commit 3528eef into master Feb 13, 2026
6 checks passed
@automergerpr-permission-manager automergerpr-permission-manager bot deleted the optimize-subst branch February 13, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants