[PATCH] D11277: [lib/Fuzzer] Add sanitizer runtime errors unit save option

Kostya Serebryany kcc at google.com
Fri Jul 17 13:53:22 PDT 2015


kcc added a comment.

In http://reviews.llvm.org/D11277#206781, @skomski wrote:

> In http://reviews.llvm.org/D11277#206713, @kcc wrote:
>
> > First question: why is a log file not enough? 
> >  It will contain the full sanitizer report (not just summary) and the reproducer. 
> >  Frankly, to me this looks like a duplicated functionality.
>
>
>
>
> In http://reviews.llvm.org/D11277#206713, @kcc wrote:
>
> > First question: why is a log file not enough? 
> >  It will contain the full sanitizer report (not just summary) and the reproducer. 
> >  Frankly, to me this looks like a duplicated functionality.
>
>
> Can you not say the same thing for the current written timeout- and crash files? I like having the files right away it makes it more convenient.


That's a bit different. The timeout- and crash- files can be directly given back to the fuzzer (the target function). 
The file with the log is for a human being to analyze -- the same as for the error report.


http://reviews.llvm.org/D11277







More information about the llvm-commits mailing list