[all-commits] [llvm/llvm-project] 52b8e1: [libclang] Remove LIBCLANG_INCLUDE_CLANG_TOOLS_EXTRA

[Nico Weber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 52b8e10597315a96bc7cbc7cfe618e301c1e6e6c
      https://github.com/llvm/llvm-project/commit/52b8e10597315a96bc7cbc7cfe618e301c1e6e6c
  Author: Nico Weber <thakis at chromium.org>
  Date:   2021-03-01 (Mon, 01 Mar 2021)

  Changed paths:
    M clang-tools-extra/test/CMakeLists.txt
    R clang-tools-extra/test/clang-tidy/infrastructure/nolint-plugin.cpp
    R clang-tools-extra/test/clang-tidy/infrastructure/nolintnextline-plugin.cpp
    M clang-tools-extra/test/lit.site.cfg.py.in
    M clang/docs/ReleaseNotes.rst
    M clang/tools/libclang/CIndex.cpp
    M clang/tools/libclang/CMakeLists.txt
    M llvm/utils/gn/secondary/clang-tools-extra/test/BUILD.gn
    M llvm/utils/gn/secondary/clang/tools/libclang/BUILD.gn
    R llvm/utils/gn/secondary/clang/tools/libclang/include_clang_tools_extra.gni

  Log Message:
  -----------
  [libclang] Remove LIBCLANG_INCLUDE_CLANG_TOOLS_EXTRA

LIBCLANG_INCLUDE_CLANG_TOOLS_EXTRA causes clang-tools-extra tools
to be included in libclang, which caused a dependency cycle. The option
has been off by default for two releases now, and (based on a web search
and mailing list feedback) nobody seems to turn it on. Remove it, like
planned on https://reviews.llvm.org/D79599

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