dkrupp requested changes to this revision.
dkrupp added a comment.
This revision now requires changes to proceed.

Thanks Gabor for writing this. 
I suggested some minor changes to the txt. Otherwise LGTM.



================
Comment at: clang/docs/analyzer/user-docs/CrossTranslationUnit.rst:98
+
+This manual procedure is boring and error-prone, so sooner or later we'd like 
to have a script which automates this for us.
+
----------------
since CodeChecker automation is already available I suggest rephrasing.

This manual procedure is error-prone and not scalable, so for analyzing real 
projects it is recommended to use the built-in support of  CodeChecker or 
scan-build-py.


================
Comment at: clang/docs/analyzer/user-docs/CrossTranslationUnit.rst:142
+The `plist` files contain the results of the analysis, which may be viewed 
with the regular analysis tools.
+E.g. one may use `CodeChecker server` and `CodeChecker store` to store and 
view the results in a web browser.
+
----------------
I suggest to replace this line with the following:
E.g. one may use CodeChecker parse to view the results in command line or 
CodeChecker parse -e html to export them into HTML format.

pure command line usage without worrying about the server setup better fits 
this command line use-case.


================
Comment at: clang/docs/analyzer/user-docs/CrossTranslationUnit.rst:147
+We actively develop CTU with CodeChecker as a "runner" script, `scan-build` is 
not actively developed for CTU.
+`scan-build` has different errors and issues, expect it to work with the very 
basic projects only.
----------------
Some basic usage description could be added still...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64801/new/

https://reviews.llvm.org/D64801



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

Reply via email to