This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Reduce complette unrolling & peeling limits
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Dominique Dhumieres <dominiq at lps dot ens dot fr>
- Cc: gcc-patches at gcc dot gnu dot org, ebotcazou at adacore dot com, hubicka at ucw dot cz
- Date: Wed, 21 Nov 2012 17:27:03 +0100
- Subject: Re: Reduce complette unrolling & peeling limits
- References: <20121118133535.4A0283BABA@mailhost.lps.ens.fr>
> > Did you notice that gcc.c-torture/compile/pr43186.c regressed? It now again
> > takes a while to compile, so times out on slow machines:
> > ...
>
> On a 2.5Ghz Core2Duo, compiling the test with revision 192891 (2012-10-28)
> takes a small fraction of a second, while with revision 193270 (2012-11-06)
> it takes ~25s.
>
> However this patch makes gfortran.dg/reassoc_4.f to fail
>
> FAIL: gfortran.dg/reassoc_4.f -O scan-tree-dump-times reassoc1 "[0-9] \\\\* " 22
>
> After it 22 should be replaced with 16 (thresshold max-completely-peeled-insns=138
> gives 16, =139 gives 22).
I would propose the following patch instead. The patch anyway changes the limits on some
targets, so lets change them on all.
Honza
Index: reassoc_4.f
===================================================================
--- reassoc_4.f (revision 193698)
+++ reassoc_4.f (working copy)
@@ -1,7 +1,5 @@
! { dg-do compile }
-! { dg-options "-O3 -ffast-math -fdump-tree-reassoc1" }
-! { dg-additional-options "--param max-completely-peel-times=16" { target spu-*-* } }
-! { dg-additional-options "--param max-completely-peeled-insns=400" { target s390*-*-* } }
+! { dg-options "-O3 -ffast-math -fdump-tree-reassoc1 --param max-completely-peel-times=16 --param max-completely-peeled-insns=400" }
subroutine anisonl(w,vo,anisox,s,ii1,jj1,weight)
integer ii1,jj1,i1,iii1,j1,jjj1,k1,l1,m1,n1
real*8 w(3,3),vo(3,3),anisox(3,3,3,3),s(60,60),weight
>
> Dominique