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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 31 01:44:37 PST 2016


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





More information about the cfe-commits mailing list