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

[Renato Golin via llvm-commits]

rengolin added inline comments.


================
Comment at: SingleSource/Benchmarks/Polybench/linear-algebra/kernels/2mm/2mm.c:140
+         DATA_TYPE POLYBENCH_2D(D_Strict_FP,NI,NL,ni,nl)) {
+#pragma STDC FP_CONTRACT OFF
+  int i, j;
----------------
why do you need the pragma here, too?


================
Comment at: SingleSource/Benchmarks/Polybench/linear-algebra/kernels/2mm/2mm.c:146
+        /* FP_ABSTOLERANCE will be added to the defines when compiling this benchmark.  */
+#define FP_ABSTOLERANCE 0.0001
+        double V1 = D[i][j];
----------------
If this has to be defined at compilation time, do we need to re-refine here? Maybe it's better to get a compilation error, to make sure the users know what they're getting.


================
Comment at: SingleSource/Benchmarks/Polybench/linear-algebra/kernels/2mm/2mm.c:208
+
+  if (check_FP(ni, nl, POLYBENCH_ARRAY(D), POLYBENCH_ARRAY(D_Strict_FP)))
+    return 1;
----------------
Why not do this to all arrays, not just D?


https://reviews.llvm.org/D25346