[PATCH] D28712: [test-release.sh] Add Polly to the list of projects

[Tobias Grosser via Phabricator via llvm-commits]

grosser added a comment.

Hi,

this is a great change.

Regarding the default off flag, Even with this flag to on, Polly will not influence the default behavior of clang/LLVM/... So the only thing that is tested by default is that our unit tests work on all platforms and that the build system does not cause troubles.

Now, I still think that we should not force Polly on our release testers. So off sounds like the right default for this patch.

Obviously, I would very much appreciate them to test Polly, but as Hans suggested we should kindly ask this on the release testers mailing list.

Best,
Tobias


https://reviews.llvm.org/D28712