[PATCH] D66030: [libFuzzer] Merge: print feature coverage number as well.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 10:19:21 PDT 2019


morehouse added inline comments.


================
Comment at: lib/fuzzer/FuzzerMerge.cpp:215
+    this->PrintStats(Where, "\n", 0, AllFeatures.size());
+  };
   Set<const TracePC::PCTableEntry *> AllPCs;
----------------
Should this use `UniqFeatures`?


================
Comment at: lib/fuzzer/FuzzerMerge.cpp:239
       if (AllFeatures.insert(Feature).second)
         UniqFeatures.insert(Feature);
     });
----------------
Hm.  Doesn't this mean `AllFeatures == UniqFeatures`?


Repository:
  rCRT Compiler Runtime

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66030/new/

https://reviews.llvm.org/D66030





More information about the llvm-commits mailing list