[PATCH] D111100: enable plugins for clang-tidy

Jameson Nash via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 31 08:24:18 PST 2022


vtjnash added a comment.

It looks like this is probably the first time that a test was written for a PLUGIN_TOOL, outside of the docs, so we are likely in new territory here :/

There seem to be a couple of paths forward:

1. disable the PLUGIN_TOOL option in cmake if the user has disabled the headers needed to make that usable to cmake
2. define the headers target unconditionally (as done by https://github.com/llvm/llvm-project/blob/ab3b89855c5318f0009e1f016ffe5b1483507fd0/clang/lib/Headers/CMakeLists.txt#L232)
3. disable the added test if the headers required to use plugins are disabled by the user


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111100/new/

https://reviews.llvm.org/D111100



More information about the cfe-commits mailing list