Now, the mutation statistics are displayed in a concise manner, along with the usefulness percentages below.
Hey Kode, is this ready for review, or you've just uploaded it for yourself? Next time, just leave a comment like "please take a look" or "PTAL" or "ready for review", so Jonathan and I can look at it without asking that question. Otherwise, we usually assume that it's a temporary version that may not necessary need our attention.
Another comment would be, please format commit title and description to be more consistent for other libFuzzer commits. Take a look at other commit messages for an example: https://github.com/llvm-mirror/compiler-rt/commits/master/lib/fuzzer
I assumed I was supposed to review this. +1 to Max's comment about saying "PTAL"
I don't really think this should be its own option. I'd just print the percentages, but I think this is for Max to decide since he has in mind what will be done with the stats better than I.
I think it would better if PrintFinalStats handled printing of all stats including the mutation ones, instead of calling these three functions all over.
This looks like an unnecessary change, please undo
Fair point. Yeah, let's print both using the same flag (i.e. PrintMutationStats). At the end of the day, I think, we may remove the detailed information with mutation names at all, but it might be useful in the meantime while we're testing the things.