[PATCH] D111100: enable plugins for clang-tidy

Tom Stellard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 16 15:27:12 PST 2022


tstellar added inline comments.


================
Comment at: clang-tools-extra/test/CMakeLists.txt:91
+  if(TARGET CTTestTidyModule)
+      list(APPEND CLANG_TOOLS_TEST_DEPS CTTestTidyModule LLVMHello)
+      target_include_directories(CTTestTidyModule PUBLIC BEFORE "${CLANG_TOOLS_SOURCE_DIR}")
----------------
Our stand-alone builds for Fedora are still not working after this patch, even with D119199.  Why is it necesary to add LLVMHello as a test dependency?  Which test is using it?


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