Details
Diff Detail
Event Timeline
This is very cool, thanks! 
I've made several minor comments. 
The patch does not apply to ToT (arc patch D19585 says 2 out of 14 hunks FAILED), for some reason, you may need to svn up / merge.
| docs/LibFuzzer.rst | ||
|---|---|---|
| 2 | For no particular reason I prefer libFuzzer. | |
| 20 | via a specific fuzzing entrypoint (aka "target function"); | |
| 21 | s/generations/generates | |
| 30 | I don't want to be this verbose.  | |
| 163 | (unless you disable this by `-reload=0` command line flag) | |
| 186 | Remove outer () | |
| 305 | s/blocks/blocks or edges | |
| 376 | why not svn? | |
| 629 | While you are at it, remove this part (about function-scope statics).  | |
Sorry, forgot to merge up first. BTW, does Phabricator show whether the patch applies cleanly somewhere? I can't immediately see it in the UI.
| docs/LibFuzzer.rst | ||
|---|---|---|
| 2 | OK, but how do you want to handle sentences that start with the name -- capitalize or not? (At the moment I've left sentence starts as "LibFuzzer does xyz...") | |
| 376–380 | When I tried it, the current SVN tip wouldn't build, so I thought it safer to stick with a fixed release -- less vulnerable that way. | |
For no particular reason I prefer libFuzzer.