Fix PR41879 (large mgrid regression)
Michael Matz
matz@suse.de
Wed Jan 20 12:54:00 GMT 2010
Hi,
On Wed, 20 Jan 2010, Richard Guenther wrote:
> > The new test fails on Linux/ia32:
> >
> > FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times
> > optimized "vect_var[^\n]*\+ " 13
> >
> > Does it need -mfpmath=sse?
>
> I would have thought
>
> ! { dg-require-effective-target vect_double }
>
> takes care of that?
It doesn't because I overwrite the options (which would have -msse2 on
x86) with dg-options, and there's no way to just _add_ to the current
options, except via implementing something like add_options_for_ieee (just
for vectorizer options). Pff, I think I'm going to checkin this patch,
making the test active only on x86(-64), so that I can hardcode -msse2.
Ciao,
Michael.
--
* gfortran.dg/vect/fast-math-mgrid-resid.f: Limit to x86, add
-msse2.
Index: testsuite/gfortran.dg/vect/fast-math-mgrid-resid.f
===================================================================
--- testsuite/gfortran.dg/vect/fast-math-mgrid-resid.f (revision 156043)
+++ testsuite/gfortran.dg/vect/fast-math-mgrid-resid.f (working copy)
@@ -1,6 +1,7 @@
-! { dg-do compile }
+! { dg-do compile { target i?86-*-* x86_64-*-* } }
! { dg-require-effective-target vect_double }
-! { dg-options "-O3 -ffast-math -fpredictive-commoning -ftree-vectorize -fdump-tree-optimized" }
+! { dg-options "-O3 -ffast-math -msse2 -fpredictive-commoning -ftree-vectorize -fdump-tree-optimized" }
+
******* RESID COMPUTES THE RESIDUAL: R = V - AU
*
More information about the Gcc-patches
mailing list