Skip to content

Enable auto_initial_contents by default in fuzz_opt.py #5943

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

tlively
Copy link
Member

@tlively tlively commented Sep 14, 2023

This setting is useful enough that there is basically no reason not to use it.
Turn it on by default to save some typing when running the fuzzer.

This setting is useful enough that there is basically no reason not to use it.
Turn it on by default to save some typing when running the fuzzer.
@tlively tlively requested a review from kripken September 14, 2023 22:23
@tlively
Copy link
Member Author

tlively commented Sep 14, 2023

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #5943 (3dfca42) into main (11dba9b) will increase coverage by 0.09%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5943      +/-   ##
==========================================
+ Coverage   42.61%   42.70%   +0.09%     
==========================================
  Files         484      485       +1     
  Lines       74831    74945     +114     
  Branches    11922    11955      +33     
==========================================
+ Hits        31886    32009     +123     
+ Misses      39750    39732      -18     
- Partials     3195     3204       +9     

see 13 files with indirect coverage changes

@tlively tlively merged commit b2e9227 into main Sep 14, 2023
@tlively tlively deleted the default-auto-contents branch September 14, 2023 23:42
radekdoulik pushed a commit to dotnet/binaryen that referenced this pull request Jul 12, 2024
)

This setting is useful enough that there is basically no reason not to use it.
Turn it on by default to save some typing when running the fuzzer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants