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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
