[PATCH] D12850: [Polly] Run polly-check-format with unit tests

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 11:04:04 PDT 2015


LGTM

> Meinersbur created this revision.
> Meinersbur added reviewers: grosser, jdoerfert.
> Meinersbur added subscribers: llvm-commits, pollydev.
> Meinersbur added a project: Polly.
>
> Add polly-check-format as dependency of check-polly
>
> http://reviews.llvm.org/D12850
>
> Files:
>    test/CMakeLists.txt
>
> Index: test/CMakeLists.txt
> ===================================================================
> --- test/CMakeLists.txt
> +++ test/CMakeLists.txt
> @@ -78,6 +78,14 @@
>
>     set_target_properties(check-polly PROPERTIES FOLDER "Polly")
>
> +  # Run polly-check-format as part of polly-check only we are compiling with

IF we are

> +  # clang, so clang-format is availbale
> +  # if (TARGET clang-format) would be preferable, but this target is only added
> +  # after Polly, i.e. there is no such target yet at this point.
> +  if (LLVM_EXTERNAL_CLANG_BUILD)
> +    add_dependencies(check-polly polly-check-format)
> +  endif ()
> +
>   endif (NOT DEFINED LLVM_MAIN_SRC_DIR)
>
>   configure_file(
>
>



More information about the llvm-commits mailing list