-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fuzzing #168
Fuzzing #168
Conversation
3262646
to
c552f13
Compare
c552f13
to
b752836
Compare
b752836
to
2f3ef59
Compare
I addressed the feedback. Please review the updates while I'm working on |
2f3ef59
to
5f0bf82
Compare
Please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like the smaller seed input as opposed to the large folder of corpus samples!
A few snippets I noticed are not present in there, some of which are new syntax and some of which are existing:
- while loops are not present
- Returning values from if-statements, as in
let x = if cond_var { 3 } else { 4 };
- repeat-loop without a fixup block
- return or fail used as nested expressions instead of just as block-level statements
- ternary update expression outside of assign-update, ie:
let x = array w/ index <- value;
There are probably more, but those few jumped to the top of my head as things we might want to include so fuzzing has a change to reuse them in other places.
3fb3a65
to
efc00a3
Compare
Please review again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The approach of including the fuzz folder in the top-level workspace is not viable and needs to be reverted.
f358773
to
4b4b524
Compare
32923ab
to
3d4c6f1
Compare
0a1467c
to
ec4229c
Compare
CR change. Cleanup. Stefan's proposed changes. Exclusded fuzzing from defaults. Update after rebase. . . . Renamed fuzz target. Added readme section, minor clean-up. Initial merge of the compiler fuzzing.
f320dab
to
9b66fdf
Compare
Adds fuzzing and code coverage during it to the repo.
Fixes #169, #257.