Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/Fuzzer/FuzzerLoop.cpp | ||
---|---|---|
382 ↗ | (On Diff #57401) | Probably unimportant here. |
lib/Fuzzer/FuzzerFlags.def | ||
---|---|---|
87 ↗ | (On Diff #58150) | I'd prefer to have this flag of by default for now |
lib/Fuzzer/FuzzerLoop.cpp | ||
362 ↗ | (On Diff #58150) | no {}, same below |
379 ↗ | (On Diff #58150) | do you expect duplicates here? |
lib/Fuzzer/test/FuzzerUnittest.cpp | ||
16 ↗ | (On Diff #58150) | make it static and not extern "C" |
Comment Actions
review
lib/Fuzzer/FuzzerLoop.cpp | ||
---|---|---|
379 ↗ | (On Diff #58150) | There might be, but I don't really care. For big corpus there would be only one point. |
Comment Actions
LGTM, ok as an off-by-default feature.
I am still not convinced it's good enough yet, will need to play and see.
Some things to check:
- are there many duplicate sizes?
- will this blow up the corpus too much?
Comment Actions
My plan is to enable it for fuzzer in chrome with big units and see if units get smaller over time.