This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Run polly-check-format with unit tests
ClosedPublic

Authored by Meinersbur on Sep 14 2015, 10:37 AM.

Details

Summary

Add polly-check-format as dependency of check-polly if clang-format is available in the same build.

Diff Detail

Repository
rL LLVM

Event Timeline

Meinersbur retitled this revision from to [Polly] Run polly-check-format with unit tests.
Meinersbur updated this object.
Meinersbur added reviewers: grosser, jdoerfert.
Meinersbur added a project: Restricted Project.
Meinersbur updated this object.
Meinersbur added subscribers: llvm-commits, pollydev.
grosser edited edge metadata.Sep 14 2015, 11:04 AM
grosser added a subscriber: grosser.

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(
Meinersbur edited edge metadata.

Add missing if

This revision was automatically updated to reflect the committed changes.