[PATCH] D12837: [Polly] Revise polly-{update|check}-format targets

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 09:36:48 PDT 2015


Meinersbur added a comment.

In http://reviews.llvm.org/D12837#245090, @grosser wrote:

> Looking at this, it would also be great if we would call polly-check-format from check-polly, if clang-format is available in our build (and we are not just a LLVM+Polly build).


In another commit.

I personally think this would be exaggerated.  Some white space in the source has no influence on the correctness of build product. Also, LLVM doesn't do this either.


http://reviews.llvm.org/D12837





More information about the llvm-commits mailing list