[clang] [clang] Register all LLVM targets in AllClangUnitTest main (PR #144428)
kadir çetinkaya via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 17 23:52:50 PDT 2025
================
@@ -117,6 +117,7 @@ get_property(LINK_LIBS GLOBAL PROPERTY CLANG_UNITTEST_LINK_LIBS)
get_property(LLVM_COMPONENTS GLOBAL PROPERTY CLANG_UNITTEST_LLVM_COMPONENTS)
add_distinct_clang_unittest(AllClangUnitTests
${SRCS}
+ AllClangUnitTests.cpp
----------------
kadircet wrote:
i don't feel great about relying on linking order and linker not loading llvm_gtest_main solely because it wasn't referenced, but I guess that's a theoretical concern that won't matter much in practice (since `llvm_gtest_main` doesn't really define symbols apart from `TestMainArgv0`, at least yet). so leaving this up to you.
but can we at least leave a comment here, mentioning that we're also linking against `llvm_gtest_main` to make it easier for people to investigate if they encounter any such issues?
https://github.com/llvm/llvm-project/pull/144428
More information about the cfe-commits
mailing list