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

[Sebastian Pop via llvm-commits]

sebpop added inline comments.


================
Comment at: SingleSource/Benchmarks/Polybench/linear-algebra/kernels/2mm/2mm.c:116
+{
+#pragma STDC FP_CONTRACT OFF
+  int i, j, k;
----------------
rengolin wrote:
> sigh... We've been over this...
What do you mean we've been over this? Please be more precise in your review.

I hope you do understand that this pragma only applies to the current function.
The test is thus testing both modes: kernel_2mm() tests with fp-contract=on and kernel_2mm_StrictFP tests with fp-contract=off.
 


https://reviews.llvm.org/D25346