[PATCH] D20301: [libfuzzer] Trying random unit prefixes during corpus load.

[Kostya Serebryany via llvm-commits]

kcc added a comment.

I wonder if it makes sense to add a test here?


================
Comment at: lib/Fuzzer/FuzzerLoop.cpp:360
@@ +359,3 @@
+// to sample short lengths much more heavily.
+void Fuzzer::TruncateUnits(std::vector<Unit> *NewCorpus) {
+  size_t MaxCorpusLen = 0;
----------------
didn't you want to flush the coverage here? 

================
Comment at: lib/Fuzzer/FuzzerLoop.cpp:382
@@ +381,3 @@
+    for (const auto &U : Corpus) {
+      if (S <= U.size() && RunOne(U.data(), S)) {
+        Unit U1(U.begin(), U.begin() + S);
----------------
S < U.size() ? 


http://reviews.llvm.org/D20301