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

[Matt Morehouse via Phabricator via llvm-commits]

morehouse added inline comments.


================
Comment at: lib/fuzzer/FuzzerLoop.cpp:369
+  if (Options.PrintMutationUsefulness)
+    MStats->PrintMutationUsefulness();
 }
----------------
kodewilliams wrote:
> morehouse wrote:
> > Should this go before the short-circuit above?  Otherwise these stats won't be printed if `PrintFinalStats` is false.
> Mutation stats relies on final stats. This way, the same line doesnt have to be put into the code 10 times (like PrintFinalStats() is).
I don't think moving this above the short circuit should affect that.  It will just make your flag have the same behavior as `-print_coverage`, `-dump_coverage`, etc. which don't require `-print_final_stats=1` to work.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D48054