diff options
| author | Eugene Zelenko <eugene.zelenko@gmail.com> | 2016-08-17 23:20:00 +0000 |
|---|---|---|
| committer | Eugene Zelenko <eugene.zelenko@gmail.com> | 2016-08-17 23:20:00 +0000 |
| commit | c139a1c252f7d4b234bab8182956b1f78aa2952b (patch) | |
| tree | 5086c03a869b67b06dd139c716fc414ba4ab6bd4 | |
| parent | 99e765b4f81247840a6cff40021c9f5de6e5e832 (diff) | |
| download | bcm5719-llvm-c139a1c252f7d4b234bab8182956b1f78aa2952b.tar.gz bcm5719-llvm-c139a1c252f7d4b234bab8182956b1f78aa2952b.zip | |
[Documentation] Remove duplicated checks groups descriptions from clang-tidy/index.rst.
Differential revision: https://reviews.llvm.org/D23596
llvm-svn: 279006
| -rw-r--r-- | clang-tools-extra/docs/clang-tidy/index.rst | 32 |
1 files changed, 9 insertions, 23 deletions
diff --git a/clang-tools-extra/docs/clang-tidy/index.rst b/clang-tools-extra/docs/clang-tidy/index.rst index 0a97b47060a..6f57bb90c4b 100644 --- a/clang-tools-extra/docs/clang-tidy/index.rst +++ b/clang-tools-extra/docs/clang-tidy/index.rst @@ -48,6 +48,8 @@ The ``-list-checks`` option lists all the enabled checks. When used without available checks or with any other value of ``-checks=`` to see which checks are enabled by this value. +:: _checks-groups-table: + There are currently the following groups of checks: ====================== ========================================================= @@ -338,29 +340,13 @@ style used in the project. For code reviews we mostly use `LLVM Phabricator`_. .. _LLVM Coding Standards: http://llvm.org/docs/CodingStandards.html .. _LLVM Phabricator: http://llvm.org/docs/Phabricator.html - -Next, you need to decide which module the check belongs to. If the check -verifies conformance of the code to a certain coding style, it probably deserves -a separate module and a directory in ``clang-tidy/``. There are already modules -implementing checks related to: - -* `C++ Core Guidelines - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/cppcoreguidelines/>`_ -* `CERT Secure Coding Standards - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/cert/>`_ -* `Google Style Guide - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/google/>`_ -* `LLVM Style - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/llvm/>`_ -* `modernizing C/C++ code - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/modernize/>`_ -* potential `performance problems - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/performance/>`_ -* various `readability issues - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/readability/>`_ -* and `miscellaneous checks - <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/misc/>`_ - that we couldn't find a better category for. +Next, you need to decide which module the check belongs to. Modules +are located in subdirectories of +``clang-tidy/ <http://reviews.llvm.org/diffusion/L/browse/clang-tools-extra/trunk/clang-tidy/>``_ +and contain checks targeting a certain aspect of code quality (performance, +readability, etc.), certain coding style or standard (Google, LLVM, CERT, etc.) +or a widely used API (e.g. MPI). Their names are same as user-facing check +groups names described :ref:`above <checks-groups-table>`. After choosing the module, you need to create a class for your check: |

