[PATCH] D25346: [test-suite] [Polybench] run tests twice with -ffp-contract=on/off

Sebastian Pop via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 02:46:35 PDT 2016


sebpop added a comment.

In https://reviews.llvm.org/D25346#564243, @rengolin wrote:

> In https://reviews.llvm.org/D25346#564231, @sebpop wrote:
>
> > > clang-format
> >
> > No.  Not in the test-suite.  We do not own that code, and I will not run clang-format on all the tests in the test-suite.
>
>
> Not in the whole test, just in your changes. You're probably using TABS or something, and the formatting is coming wrong. Please look at the result in this page and you'll see.


I will run clang-format-patch on the changes in the patch.


https://reviews.llvm.org/D25346





More information about the llvm-commits mailing list