[all-commits] [llvm/llvm-project] 462d4d: [clangd] Add CMake option to (not) link in clang-t...

[Sam McCall via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 462d4de35b0c9ef7157e49e147fc448a40c829b1
      https://github.com/llvm/llvm-project/commit/462d4de35b0c9ef7157e49e147fc448a40c829b1
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2021-07-14 (Wed, 14 Jul 2021)

  Changed paths:
    M clang-tools-extra/clangd/CMakeLists.txt
    M clang-tools-extra/clangd/Features.cpp
    M clang-tools-extra/clangd/Features.inc.in
    M clang-tools-extra/clangd/ParsedAST.cpp
    A clang-tools-extra/clangd/test/diagnostics-tidy.test
    R clang-tools-extra/clangd/test/diagnostics.test
    M clang-tools-extra/clangd/test/lit.cfg.py
    M clang-tools-extra/clangd/test/lit.site.cfg.py.in
    M clang-tools-extra/clangd/unittests/ConfigCompileTests.cpp
    M clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp

  Log Message:
  -----------
  [clangd] Add CMake option to (not) link in clang-tidy checks

This reduces the size of the dependency graph and makes incremental
development a little more pleasant (less rebuilding).

This introduces a bit of complexity/fragility as some tests verify
clang-tidy behavior. I attempted to isolate these and build/run as much
of the tests as possible in both configs to prevent rot.

Expectation is that (some) developers will use this locally, but
buildbots etc will keep testing clang-tidy.

Fixes https://github.com/clangd/clangd/issues/233

Differential Revision: https://reviews.llvm.org/D105679