Skip to content

Commit bbb8671

Browse files
committedMay 15, 2017
[Polly][CMake] Exclude isl_config from the polly-check-format target.
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
1 parent aa2b7a6 commit bbb8671

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎polly/CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ configure_file( ${CMAKE_CURRENT_SOURCE_DIR}/include/polly/Config/config.h.cmake
244244

245245
# Add target to check formatting of polly files
246246
file( GLOB_RECURSE files *.h lib/*.cpp lib/*.c tools/*.cpp tools/*.c tools/*.h unittests/*.cpp)
247-
file( GLOB_RECURSE external lib/External/*.h lib/External/*.c lib/External/*.cpp)
247+
file( GLOB_RECURSE external lib/External/*.h lib/External/*.c lib/External/*.cpp isl_config.h)
248248
list( REMOVE_ITEM files ${external})
249249

250250
set(check_format_depends)

0 commit comments

Comments
 (0)
Please sign in to comment.