[PATCH] D64801: [analyzer] Add CTU user docs
Daniel Krupp via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 17 02:54:15 PDT 2019
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
More information about the cfe-commits
mailing list