[PATCH] D137890: Add install targets for gtest
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 22 05:15:37 PST 2022
mgorny added inline comments.
================
Comment at: third-party/unittest/CMakeLists.txt:81
+ ARCHIVE DESTINATION "lib${LLVM_LIBDIR_SUFFIX}" COMPONENT llvm_gtest)
+ install(EXPORT LLVMGTestConfig DESTINATION ${LLVM_INSTALL_PACKAGE_DIR} COMPONENT llvm_gtest)
+ add_llvm_install_targets(install-llvm_gtest COMPONENT llvm_gtest DEPENDS llvm_gtest LLVMGTestConfig.cmake)
----------------
Any reason you think this should be installed separately rather than included in the usual LLVM export list? I think the latter would make it more consistent to use in in-tree vs standalone builds (as you wouldn't have to `find_package` it).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137890/new/
https://reviews.llvm.org/D137890
More information about the llvm-commits
mailing list