[libFuzzer] minor refactoring, NFC
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk/lib/fuzzer@313406 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/FuzzerDriver.cpp b/FuzzerDriver.cpp
index 804f426..cc99534 100644
--- a/FuzzerDriver.cpp
+++ b/FuzzerDriver.cpp
@@ -471,9 +471,8 @@
// Get coverage for the testcase without modifications.
F->ExecuteCallback(C.data(), C.size());
InitialFeatures.clear();
- TPC.CollectFeatures([&](size_t Feature) -> bool {
+ TPC.CollectFeatures([&](size_t Feature) {
InitialFeatures.push_back(Feature);
- return true;
});
for (size_t i = 0; i < Dict.size(); ++i) {
@@ -498,9 +497,8 @@
// Get coverage for testcase with masked occurrences of dictionary unit.
F->ExecuteCallback(Data.data(), Data.size());
ModifiedFeatures.clear();
- TPC.CollectFeatures([&](size_t Feature) -> bool {
+ TPC.CollectFeatures([&](size_t Feature) {
ModifiedFeatures.push_back(Feature);
- return true;
});
if (InitialFeatures == ModifiedFeatures)
diff --git a/FuzzerMerge.cpp b/FuzzerMerge.cpp
index 78f2253..03cf00a 100644
--- a/FuzzerMerge.cpp
+++ b/FuzzerMerge.cpp
@@ -241,10 +241,9 @@
// So it makes no sense to record all features for all files, instead we
// only record features that were not seen before.
Set<size_t> UniqFeatures;
- TPC.CollectFeatures([&](size_t Feature) -> bool {
+ TPC.CollectFeatures([&](size_t Feature) {
if (AllFeatures.insert(Feature).second)
UniqFeatures.insert(Feature);
- return true;
});
// Show stats.
if (!(TotalNumberOfRuns & (TotalNumberOfRuns - 1)))
diff --git a/FuzzerTracePC.h b/FuzzerTracePC.h
index 5417260..743db54 100644
--- a/FuzzerTracePC.h
+++ b/FuzzerTracePC.h
@@ -208,7 +208,7 @@
return Bit;
}
-template <class Callback> // bool Callback(size_t Feature)
+template <class Callback> // void Callback(size_t Feature)
ATTRIBUTE_NO_SANITIZE_ADDRESS
__attribute__((noinline))
void TracePC::CollectFeatures(Callback HandleFeature) const {