[PATCH] D131919: Move googletest to the third-party directory

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 10:39:36 PDT 2022


probinson added inline comments.


================
Comment at: clang/CMakeLists.txt:106
           AND EXISTS ${UNITTEST_DIR}/CMakeLists.txt)
         add_subdirectory(${UNITTEST_DIR} utils/unittest)
       endif()
----------------
Should this be `third-party/unittest` ? Looking at how the lldb cmake files were modified.


================
Comment at: lld/CMakeLists.txt:69
           AND EXISTS ${UNITTEST_DIR}/CMakeLists.txt)
         add_subdirectory(${UNITTEST_DIR} utils/unittest)
       endif()
----------------
See comment for clang/CMakeLists.txt


================
Comment at: mlir/CMakeLists.txt:25
   if(EXISTS ${UNITTEST_DIR}/googletest/include/gtest/gtest.h)
     add_subdirectory(${UNITTEST_DIR} utils/unittest)
   endif()
----------------
See comment for clang/CMakeLists.txt


================
Comment at: polly/CMakeLists.txt:37
     if (NOT TARGET gtest)
       add_subdirectory(${UNITTEST_DIR} utils/unittest)
     endif()
----------------
See comment for clang/CMakeLists.txt


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131919/new/

https://reviews.llvm.org/D131919



More information about the cfe-commits mailing list