[PATCH] D28189: Extend documentation of how to test clang-tidy checks.

[Alexander Kornienko via Phabricator via cfe-commits]

alexfh added inline comments.


================
Comment at: docs/clang-tidy/index.rst:558
+typically the basic `CHECK` forms (`CHECK-MESSAGES` and `CHECK-FIXES`)
+are sufficient for matcher tests.  Note that the `FileCheck`
+documentation mostly assumes the default prefix (`CHECK`), and hence
----------------
I'm not sure the term "matcher tests" is used anywhere else in our documentation. And it doesn't seem obvious to me either (if it refers to AST matchers, then it's just an implementation detail of clang-tidy checks, if it refers to the check patterns, then it's not a commonly used term for those either). I don't know what would be a good short replacement, maybe "clang-tidy lit tests" or just "clang-tidy tests".


https://reviews.llvm.org/D28189