[PATCH] D111100: enable plugins for clang-tidy
Tom Stellard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 17 09:01:44 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}")
----------------
aaron.ballman wrote:
> tstellar wrote:
> > 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?
> CTestTidyModule.cpp is using it:
> ```
> // RUN: clang-tidy -checks='-*,mytest*' --list-checks -load %llvmshlibdir/CTTestTidyModule%pluginext -load %llvmshlibdir/LLVMHello%pluginext | FileCheck --check-prefix=CHECK-LIST %s
> ```
> I *think* this is testing that we can load a clang-tidy plugin and an llvm plugin at the same time and not hit conflicting symbols or other issues. If I'm correct, then I think that's useful test functionality, but I wouldn't describe it as critical, so I'd be fine if we dropped it for now to get this patch in, and then added the extra testing in a subsequent patch if we think it's necessary.
OK, that does sound like a useful test. I will spend some time investigating how to make this work with stand-alone builds.
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