[PATCH] D48054: [libFuzzer] Mutation tracking and logging implemented

[Jonathan Metzman via Phabricator via llvm-commits]

metzman added inline comments.


================
Comment at: lib/fuzzer/FuzzerMutationStats.cpp:26
+     (1 + TotalMutations.at(i));
+    Printf("%.3f", UsefulPercentage);
+    if (i < NUMBER_OF_MUTATION_TYPES - 1) Printf(",");
----------------
morehouse wrote:
> kodewilliams wrote:
> > morehouse wrote:
> > > The output will be easier to understand if you also print the mutation names.
> > I was told to remove the names from the output for parsing purposes, as it will make it easier to use in the latter phase of the project. Does this still stand?
> I was interpreting this patch as an option to print human-readable stats similar to `-print_final_stats`.
> 
> @kcc, @Dor1s:  Do we care if this information is human readable or not?
@Dor1s is OOO for 2 weeks. 
But I think he was more concerned that the output be easy for libFuzzer to consume than human readable. 
I believe he wanted it output as a CSV without any column names. 


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D48054