The issue was that we were compiling the pph with some flags and weren't doing 
so in the rest of the compilation (pph and non-pph), resulting in different 
outputs.

The fact that pph would let this happen is potentially a problem and I opened a 
separate thread "Flags and pph..." to discuss this.

We potentially want to store which flags were on at the pph compile time and 
make sure the same flags are on when doing the real compile (otherwise 
recompile the pph or some other soltution...??)

How we decide to handle this is beyond the scope of just fixing this test, so 
here is the patch for now (and I just realized I forgot to add the ChangeLog, 
but will add it as part of the commit...)

Gab

diff --git a/gcc/testsuite/g++.dg/pph/c1builtin-integral.cc 
b/gcc/testsuite/g++.dg/pph/c1builtin-integral.cc
index 6887b11..ae33f8d 100644
--- a/gcc/testsuite/g++.dg/pph/c1builtin-integral.cc
+++ b/gcc/testsuite/g++.dg/pph/c1builtin-integral.cc
@@ -1,2 +1,2 @@
-// pph asm xdiff 52758
+/* { dg-options "-ffinite-math-only -fno-math-errno" } */
 #include "c0builtin-integral.h"

--
This patch is available for review at http://codereview.appspot.com/4761045

Reply via email to