[Polly][CMake] Skip unit-tests in lit if gtest is not available
commit6370a979e02288f13504e44b0fb2a15a48e5e7bd
authorPhilip Pfaffe <philip.pfaffe@gmail.com>
Tue, 11 Jul 2017 11:37:35 +0000 (11 11:37 +0000)
committerPhilip Pfaffe <philip.pfaffe@gmail.com>
Tue, 11 Jul 2017 11:37:35 +0000 (11 11:37 +0000)
treeea2858127700fc339c550b73ff956e95f9cd9501
parent087265ab2584ba18447dd911951db9ff60630aa1
[Polly][CMake] Skip unit-tests in lit if gtest is not available

Summary:
There is a bug in the current lit configurations for the unittests. If gtest is not available, the site-config for the unit tests won't be generated. Because lit recurses through the test directory, the lit configuration for the unit tests will be discovered nevertheless, leading to a fatal error in lit.

This patch semi-gracefully skips the unittests if gtest is not available. As a result, running lit now prints this: `warning: test suite 'Polly-Unit' contained no test`.

If people think that this is too annoying, the alternative would be to pick apart the test directory, so that the lit testsuite discovery will always only find one configuration. In fact, both of these things could be combined. While it's certainly nice that running a single lit command runs all the tests, I suppose people use the `check-polly` make target over lit most of the time, so the difference might not be noticed.

Reviewers: Meinersbur, grosser

Reviewed By: grosser

Subscribers: mgorny, bollu, pollydev, llvm-commits

Tags: #polly

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

git-svn-id: https://llvm.org/svn/llvm-project/polly/trunk@307651 91177308-0d34-0410-b5e6-96231b3b80d8
CMakeLists.txt
test/CMakeLists.txt
test/Unit/lit.cfg
test/Unit/lit.site.cfg.in