[PATCH] D44078: [Polly][CMake] Fix lit setup for building the in the mono repo

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 4 22:42:02 PST 2018


This LGTM. (Seems phab is down ATM, so sending via email).

Best,
Tobias

On Sun, Mar 4, 2018, at 16:43, Philip Pfaffe via Phabricator via llvm-commits wrote:
> philip.pfaffe created this revision.
> philip.pfaffe added reviewers: Meinersbur, grosser.
> Herald added subscribers: bollu, mgorny.
> Herald added a reviewer: bollu.
> 
> When building polly as part of the monorepo (actually, as part of any setup
> using LLVM_ENABLE_PROJECTS), the LLVMPolly library used in the lit tests ends
> up in a different directory in the build tree than in an in-tree build
> 
> 
> Repository:
>   rPLO Polly
> 
> https://reviews.llvm.org/D44078
> 
> Files:
>   test/CMakeLists.txt
> 
> 
> Index: test/CMakeLists.txt
> ===================================================================
> --- test/CMakeLists.txt
> +++ test/CMakeLists.txt
> @@ -47,7 +47,11 @@
>  set(LLVM_BINARY_DIR "${LLVM_BINARY_DIR}")
>  set(LLVM_TOOLS_DIR "${LLVM_TOOLS_BINARY_DIR}")
>  set(LLVM_LIBS_DIR "${LLVM_BINARY_DIR}/lib${LLVM_LIBDIR_SUFFIX}")
> -set(POLLY_LIB_DIR "${POLLY_BINARY_DIR}/lib")
> +if (CMAKE_LIBRARY_OUTPUT_DIRECTORY)
> +  set(POLLY_LIB_DIR ${CMAKE_LIBRARY_OUTPUT_DIRECTORY})
> +else()
> +  set(POLLY_LIB_DIR "${POLLY_BINARY_DIR}/lib")
> +endif()
>  
>  configure_lit_site_cfg(
>    ${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.in
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> Email had 1 attachment:
> + D44078.136939.patch
>   1k (text/x-patch)


More information about the llvm-commits mailing list