[PATCH] D33192: [Polly][CMake] Exclude isl_config from the polly-check-format target.

Philip Pfaffe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 06:20:30 PDT 2017


philip.pfaffe created this revision.
philip.pfaffe added a project: Polly.
Herald added a subscriber: mgorny.

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.


Repository:
  rL LLVM

https://reviews.llvm.org/D33192

Files:
  CMakeLists.txt


Index: CMakeLists.txt
===================================================================
--- CMakeLists.txt
+++ CMakeLists.txt
@@ -244,7 +244,7 @@
 
 # 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)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33192.98993.patch
Type: text/x-patch
Size: 542 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170515/e0908456/attachment.bin>


More information about the llvm-commits mailing list