[polly] r308512 - [CMake] Fix r307650: Readd missing dependency.

[Hans Wennborg via llvm-commits]

Sounds good to me. Tobias: is this ok to merge?

On Thu, Jul 20, 2017 at 6:04 PM, Philip Pfaffe <philip.pfaffe at gmail.com> wrote:
> Hi Hans,
>
> I think this commit should be merged into the release branch. It fixes the
> Polly check-polly build target in case opt, FileCheck, or similar weren't
> built.
>
> Cheers,
> Philip
>
> ---------- Forwarded message ----------
> From: Philip Pfaffe via llvm-commits <llvm-commits at lists.llvm.org>
> Date: 2017-07-19 21:20 GMT+02:00
> Subject: [polly] r308512 - [CMake] Fix r307650: Readd missing dependency.
> To: llvm-commits at lists.llvm.org
>
>
> Author: pfaffe
> Date: Wed Jul 19 12:20:58 2017
> New Revision: 308512
>
> URL: http://llvm.org/viewvc/llvm-project?rev=308512&view=rev
> Log:
> [CMake] Fix r307650: Readd missing dependency.
>
> The commit erroneously removed the dependency of the Polly tests on
> things like opt and FileCheck. Add that dependency back.
>
> Modified:
>     polly/trunk/test/CMakeLists.txt
>
> Modified: polly/trunk/test/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/polly/trunk/test/CMakeLists.txt?rev=308512&r1=308511&r2=308512&view=diff
> ==============================================================================
> --- polly/trunk/test/CMakeLists.txt (original)
> +++ polly/trunk/test/CMakeLists.txt Wed Jul 19 12:20:58 2017
> @@ -28,14 +28,15 @@ if(NOT LLVM_MAIN_SRC_DIR)
>    if ("${POLLY_TEST_EXTRA_PATHS}" STREQUAL "${LLVM_TOOLS_BINARY_DIR}")
>      set(POLLY_TEST_EXTRA_PATHS "")
>    endif()
> +  set(POLLY_TEST_DEPS LLVMPolly)
>  else ()
>    set(LLVM_OPT "${LLVM_TOOLS_BINARY_DIR}/opt")
>    set(LLVM_FILECHECK "${LLVM_TOOLS_BINARY_DIR}/FileCheck")
>    set(LLVM_NOT "${LLVM_TOOLS_BINARY_DIR}/not")
>    set(POLLY_TEST_EXTRA_PATHS "")
> +  set(POLLY_TEST_DEPS llvm-config opt LLVMPolly FileCheck not)
>  endif()
>
> -set(POLLY_TEST_DEPS LLVMPolly)
>  if (POLLY_BUNDLED_ISL)
>    list(APPEND POLLY_TEST_DEPS polly-isl-test)
>  endif()
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>