From ffd22ec7569874c5c157856abc5d92ae2c277c5b Mon Sep 17 00:00:00 2001 From: Evgeny Mankov Date: Mon, 23 Oct 2017 21:16:13 +0300 Subject: [PATCH 1/3] [HIPIFY] cmake: fix standalone build --- hipify-clang/CMakeLists.txt | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/hipify-clang/CMakeLists.txt b/hipify-clang/CMakeLists.txt index 355b0cea7a..2d9cbb1feb 100644 --- a/hipify-clang/CMakeLists.txt +++ b/hipify-clang/CMakeLists.txt @@ -75,15 +75,21 @@ if (HIPIFY_CLANG_TESTS) set(BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR}) + if (CMAKE_CURRENT_SOURCE_DIR EQUAL CMAKE_SOURCE_DIR) + set(SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}) + else() + set(SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/..) + endif() + configure_file( - ${CMAKE_SOURCE_DIR}/tests/hipify-clang/lit.site.cfg.in - ${CMAKE_CURRENT_BINARY_DIR}/tests/hipify-clang/lit.site.cfg + ${SOURCE_DIR}/tests/hipify-clang/lit.site.cfg.in + ${BINARY_DIR}/tests/hipify-clang/lit.site.cfg @ONLY ) add_lit_testsuite(test-hipify "Running HIPify regression tests" - ${CMAKE_SOURCE_DIR}/tests/hipify-clang - PARAMS site_config=${CMAKE_CURRENT_BINARY_DIR}/tests/hipify-clang/lit.site.cfg + ${SOURCE_DIR}/tests/hipify-clang + PARAMS site_config=${BINARY_DIR}/tests/hipify-clang/lit.site.cfg DEPENDS hipify-clang lit ) From 062127c5ea1a8a7fa238c40e8526ac1b31a71ca3 Mon Sep 17 00:00:00 2001 From: emankov Date: Tue, 24 Oct 2017 10:51:11 +0300 Subject: [PATCH 2/3] [HIPIFY] cmake: simplify build --- hipify-clang/CMakeLists.txt | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/hipify-clang/CMakeLists.txt b/hipify-clang/CMakeLists.txt index 2d9cbb1feb..5f4f96fb8a 100644 --- a/hipify-clang/CMakeLists.txt +++ b/hipify-clang/CMakeLists.txt @@ -73,23 +73,15 @@ if (HIPIFY_CLANG_TESTS) # tests find_package(PythonInterp 2.7 REQUIRED EXACT) - set(BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR}) - - if (CMAKE_CURRENT_SOURCE_DIR EQUAL CMAKE_SOURCE_DIR) - set(SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}) - else() - set(SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/..) - endif() - configure_file( - ${SOURCE_DIR}/tests/hipify-clang/lit.site.cfg.in - ${BINARY_DIR}/tests/hipify-clang/lit.site.cfg + $${CMAKE_CURRENT_LIST_DIR}/../tests/hipify-clang/lit.site.cfg.in + ${CMAKE_CURRENT_BINARY_DIR}/tests/hipify-clang/lit.site.cfg @ONLY ) add_lit_testsuite(test-hipify "Running HIPify regression tests" - ${SOURCE_DIR}/tests/hipify-clang - PARAMS site_config=${BINARY_DIR}/tests/hipify-clang/lit.site.cfg + ${CMAKE_CURRENT_LIST_DIR}/../tests/hipify-clang + PARAMS site_config=${CMAKE_CURRENT_BINARY_DIR}/tests/hipify-clang/lit.site.cfg DEPENDS hipify-clang lit ) From 5a9d4e3435e54ecb1b6ce3c3f52d9ada1bb9b066 Mon Sep 17 00:00:00 2001 From: Evgeny Mankov Date: Tue, 24 Oct 2017 14:16:05 +0300 Subject: [PATCH 3/3] [HIPIFY][fix] cmake: do not build hipify-clang if not asked + warn "hipify-clang will not be built" if HIPIFY_CLANG_LLVM_DIR is not specified. + fix typo in previous commit . --- hipify-clang/CMakeLists.txt | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/hipify-clang/CMakeLists.txt b/hipify-clang/CMakeLists.txt index 5f4f96fb8a..b9d009ad92 100644 --- a/hipify-clang/CMakeLists.txt +++ b/hipify-clang/CMakeLists.txt @@ -1,6 +1,15 @@ cmake_minimum_required(VERSION 2.8.8) project(hipify-clang) +set(BUILD_HIPIFY_CLANG 0 CACHE INTERNAL "") + +if (HIPIFY_CLANG_LLVM_DIR) + find_package(LLVM PATHS ${HIPIFY_CLANG_LLVM_DIR} REQUIRED) +else() + message(STATUS "hipify-clang will not be built. To build it please specify absolute path to LLVM 3.8 or higher using HIPIFY_CLANG_LLVM_DIR") + return() +endif() + option(HIPIFY_CLANG_TESTS "Build the tests for hipify-clang, if lit is installed" ON) # Disable the tests if `lit` is not installed. @@ -10,10 +19,6 @@ if (NOT LIT_COMMAND) message(STATUS "hipify-clang's tests are not being built because `lit` is not installed.") endif() -set(BUILD_HIPIFY_CLANG 0 CACHE INTERNAL "") - -find_package(LLVM PATHS ${HIPIFY_CLANG_LLVM_DIR} REQUIRED) - list(APPEND CMAKE_MODULE_PATH ${LLVM_CMAKE_DIR}) include(AddLLVM) @@ -74,7 +79,7 @@ if (HIPIFY_CLANG_TESTS) find_package(PythonInterp 2.7 REQUIRED EXACT) configure_file( - $${CMAKE_CURRENT_LIST_DIR}/../tests/hipify-clang/lit.site.cfg.in + ${CMAKE_CURRENT_LIST_DIR}/../tests/hipify-clang/lit.site.cfg.in ${CMAKE_CURRENT_BINARY_DIR}/tests/hipify-clang/lit.site.cfg @ONLY )