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

[Sebastian Pop via llvm-commits]

sebpop added a comment.

In https://reviews.llvm.org/D25346#570962, @MatzeB wrote:

> LGTM, thanks for fixing this properly. I leave the final approval to Renato.


I will wait for Renato's approval before commit.



================
Comment at: SingleSource/Benchmarks/Polybench/datamining/correlation/correlation.c:137
+
+#define sqrt_of_array_cell(x,j) sqrt(x[j])
+
----------------
MatzeB wrote:
> you actually have to write more to use this macro than to just use sqrt(x[j]) in the first place.
I have not changed the code of the kernel.
I intentionally copied the full text without any change from my side in order to avoid any possible error.
If you think there is an error with this line, let's fix that in a separate patch.


https://reviews.llvm.org/D25346