[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