[compiler-rt] 4034d0c - [libFuzzer] Fixed description of fuzzer merge control file.
    Kostya Serebryany via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri May 29 16:00:44 PDT 2020
    
    
  
Author: Adrian Herrera
Date: 2020-05-29T16:00:26-07:00
New Revision: 4034d0ce207d0c545799ceae5aaf2e35d400407c
URL: https://github.com/llvm/llvm-project/commit/4034d0ce207d0c545799ceae5aaf2e35d400407c
DIFF: https://github.com/llvm/llvm-project/commit/4034d0ce207d0c545799ceae5aaf2e35d400407c.diff
LOG: [libFuzzer] Fixed description of fuzzer merge control file.
Summary:
The description of the fuzzer merge control file appears to be incorrect/out of date.
No "DONE" line appears in the control file. Rather, FT and COV are the markers that appear
following the STARTED line.
Reviewers: metzman, kcc
Reviewed By: kcc
Subscribers: #sanitizers
Tags: #sanitizers
Differential Revision: https://reviews.llvm.org/D80788
Added: 
    
Modified: 
    compiler-rt/lib/fuzzer/FuzzerMerge.h
Removed: 
    
################################################################################
diff  --git a/compiler-rt/lib/fuzzer/FuzzerMerge.h b/compiler-rt/lib/fuzzer/FuzzerMerge.h
index c14dd589e62d..e0c6bc539bdb 100644
--- a/compiler-rt/lib/fuzzer/FuzzerMerge.h
+++ b/compiler-rt/lib/fuzzer/FuzzerMerge.h
@@ -13,7 +13,7 @@
 //   The process should tolerate the crashes, OOMs, leaks, etc.
 //
 // Algorithm:
-//   The outter process collects the set of files and writes their names
+//   The outer process collects the set of files and writes their names
 //   into a temporary "control" file, then repeatedly launches the inner
 //   process until all inputs are processed.
 //   The outer process does not actually execute the target code.
@@ -22,13 +22,14 @@
 //   and b) the last processed input. Then it starts processing the inputs one
 //   by one. Before processing every input it writes one line to control file:
 //   STARTED INPUT_ID INPUT_SIZE
-//   After processing an input it write another line:
-//   DONE INPUT_ID Feature1 Feature2 Feature3 ...
+//   After processing an input it writes the following lines:
+//   FT INPUT_ID Feature1 Feature2 Feature3 ...
+//   COV INPUT_ID Coverage1 Coverage2 Coverage3 ...
 //   If a crash happens while processing an input the last line in the control
 //   file will be "STARTED INPUT_ID" and so the next process will know
 //   where to resume.
 //
-//   Once all inputs are processed by the innner process(es) the outer process
+//   Once all inputs are processed by the inner process(es) the outer process
 //   reads the control files and does the merge based entirely on the contents
 //   of control file.
 //   It uses a single pass greedy algorithm choosing first the smallest inputs
        
    
    
More information about the llvm-commits
mailing list