Skip to content

Enable auto_initial_contents by default in fuzz_opt.py#5943

Merged
tlively merged 1 commit intomainfrom
default-auto-contents
Sep 14, 2023
Merged

Enable auto_initial_contents by default in fuzz_opt.py#5943
tlively merged 1 commit intomainfrom
default-auto-contents

Conversation

@tlively
Copy link
Copy Markdown
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
Copy Markdown
Member Author

tlively commented Sep 14, 2023

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@codecov
Copy link
Copy Markdown

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.
aheejin added a commit to aheejin/binaryen that referenced this pull request Apr 21, 2026
 WebAssembly#5943 changed the default from disabled to enabled but comment was left
unchanged.
aheejin added a commit that referenced this pull request Apr 21, 2026
#5943 changed the default from disabled to enabled but comment was left
unchanged.
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