[Polly][CMake] Exclude isl_config from the polly-check-format target.
authorPhilip Pfaffe <philip.pfaffe@gmail.com>
Mon, 15 May 2017 13:20:26 +0000 (13:20 +0000)
committerPhilip Pfaffe <philip.pfaffe@gmail.com>
Mon, 15 May 2017 13:20:26 +0000 (13:20 +0000)
Summary:
The custom `polly-check-format` target runs clang-format over all source files in the directory tree excluding lib/External. `isl_config.h` is a header file that is generated by CMake in the build directory, and it's not correctly formatted (which I also wouldn't consider necessary, as it is a generated file).

If the build directory is actually inside the Polly source directory (which it might be if you're building Polly out-of-tree), that check always fails. Hence this patch excludes this file from the check-format target.

Reviewers: Meinersbur, grosser

Reviewed By: grosser

Subscribers: mgorny, llvm-commits, pollydev

Tags: #polly

Differential Revision: https://reviews.llvm.org/D33192

llvm-svn: 303060

polly/CMakeLists.txt

index 353b097..a748c22 100644 (file)
@@ -244,7 +244,7 @@ configure_file( ${CMAKE_CURRENT_SOURCE_DIR}/include/polly/Config/config.h.cmake
 
 # Add target to check formatting of polly files
 file( GLOB_RECURSE files *.h lib/*.cpp lib/*.c tools/*.cpp tools/*.c tools/*.h unittests/*.cpp)
-file( GLOB_RECURSE external lib/External/*.h lib/External/*.c lib/External/*.cpp)
+file( GLOB_RECURSE external lib/External/*.h lib/External/*.c lib/External/*.cpp isl_config.h)
 list( REMOVE_ITEM files ${external})
 
 set(check_format_depends)