[PATCH] D137890: Add install targets for gtest

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 20:55:28 PST 2022


mgorny added a comment.

I'll try to test this today. I suppose the main idea would be to enable `LLVM_INSTALL_GTEST`, then stop unpacking `third-party` directory when building other packages, correct? Perhaps it would make sense to update the checks like the one in clang while doing that:

  if( CLANG_INCLUDE_TESTS )
    if(EXISTS ${LLVM_THIRD_PARTY_DIR}/unittest/googletest/include/gtest/gtest.h)
      add_subdirectory(unittests)
      ...


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