[PATCH] D68132: clang-tidy: Don't repeat list of all checks in three places.

2019-09-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL373082: clang-tidy: Don't repeat list of all checks in three places. (authored by nico, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: ht

[PATCH] D68132: clang-tidy: Don't repeat list of all checks in three places.

2019-09-27 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Thanks! I'll land this, but beanz, if the cmake setup here is weird please shout and I'll try to make it less weird. We have a few other instances of PARENT_SCOPE variables, so maybe this is how cmake likes to do "group of libraries". CHANGES SINCE LAST ACTION https:

[PATCH] D68132: clang-tidy: Don't repeat list of all checks in three places.

2019-09-27 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added reviewers: beanz, mwyman, gribozavr. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, ilya-biryukov, mgorny, srhines. Herald added a reviewer: jdoerfert. Instead, put all checks in a cmake variable and reference this. Also, make clangd