zaks.anna accepted this revision.
zaks.anna added a comment.
This revision is now accepted and ready to land.

A couple of nits. Otherwise, LGTM. Thanks!


================
Comment at: www/analyzer/checker_dev_manual.html:534
@@ +533,3 @@
+itself to the child process (for example, in <tt><b>gcc</b></tt> you can
+<tt><b>set follow-fork-mode=child</b></tt>), or use the <tt><b>-###</b></tt>
+flag for obtaining the command line of the child process:
----------------
I suggest to just tell about -### and drop the explanation about attaching to 
the child process. Sounds good?

================
Comment at: www/analyzer/checker_dev_manual.html:645
@@ +644,3 @@
+
+<p>The bug reporter mechanism removes path diagnostics inside intermediate
+function calls that have returned by the time the bug was found and contain
----------------
I'd move this up after the description of "-analyze-function=foo".


https://reviews.llvm.org/D22874



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to