* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
@ 2013-12-13 14:31 ` dominiq at lps dot ens.fr
2013-12-19 13:34 ` rguenth at gcc dot gnu.org
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: dominiq at lps dot ens.fr @ 2013-12-13 14:31 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
Dominique d'Humieres <dominiq at lps dot ens.fr> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2013-12-13
Ever confirmed|0 |1
--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
Forgot to add the pointer to
http://gcc.gnu.org/ml/gcc-regression/2013-10/msg00205.html.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
2013-12-13 14:31 ` [Bug testsuite/59494] " dominiq at lps dot ens.fr
@ 2013-12-19 13:34 ` rguenth at gcc dot gnu.org
2013-12-20 13:29 ` jakub at gcc dot gnu.org
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-12-19 13:34 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.9.0
--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Well, I don't think configuring with arbitrary --with-cpu/tune is a supported
configuration for zero testsuite FAILs ...?
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
2013-12-13 14:31 ` [Bug testsuite/59494] " dominiq at lps dot ens.fr
2013-12-19 13:34 ` rguenth at gcc dot gnu.org
@ 2013-12-20 13:29 ` jakub at gcc dot gnu.org
2014-01-02 14:28 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-12-20 13:29 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
True, I guess adding -mtune=generic doesn't hurt though, it will be still
broken with --target_board=unix/-mtune=core2 or similar testing, but at least
it will FAIL less often. For core* tuning the difference is e.g. in reassoc
(different reassoc widths, fairly recent change), and starting with probably
r160636 different IVopts decisions based on different costs. I believe that is
the only reason for the different number of vector additions.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
` (2 preceding siblings ...)
2013-12-20 13:29 ` jakub at gcc dot gnu.org
@ 2014-01-02 14:28 ` jakub at gcc dot gnu.org
2014-01-13 12:25 ` jakub at gcc dot gnu.org
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-01-02 14:28 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Alternatively we should just remove that scan-tree-dump-times from the test
altogether, counting additions where either lhs or rhs1 of addition has vect_
in name is quite flawed, we have lots of other vector additions that just use
anonymous SSA_NAMEs without any name (_114 etc.), but still have vector types.
Counting addps etc. insns doesn't work though, there are 23 before and after
the r156043 change.
Richard, what is your preference here?
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
` (3 preceding siblings ...)
2014-01-02 14:28 ` jakub at gcc dot gnu.org
@ 2014-01-13 12:25 ` jakub at gcc dot gnu.org
2014-01-13 12:36 ` dominiq at lps dot ens.fr
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-01-13 12:25 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 31820
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31820&action=edit
gcc49-pr59494.patch
Fix that tries to capture the r156042 to r156043 changes in the *.pcom dump
details.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
` (4 preceding siblings ...)
2014-01-13 12:25 ` jakub at gcc dot gnu.org
@ 2014-01-13 12:36 ` dominiq at lps dot ens.fr
2014-01-14 9:01 ` jakub at gcc dot gnu.org
2014-01-14 9:04 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: dominiq at lps dot ens.fr @ 2014-01-13 12:36 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
--- Comment #8 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
With the patch in comment 7 the test succeeds for
Target: x86_64-apple-darwin10.8.0
Configured with: ../work/configure --prefix=/opt/gcc/gcc4.9w
--enable-languages=c,c++,fortran,objc,obj-c++,ada,java,lto --with-gmp=/opt/mp
--with-system-zlib --with-isl=/opt/mp --enable-lto --enable-plugin
and
Target: x86_64-apple-darwin13.0.0
Configured with: ../work/configure --prefix=/opt/gcc/gcc4.9w
--enable-languages=c,c++,fortran,objc,obj-c++,ada,java,lto --with-gmp=/opt/mp
--with-system-zlib --with-isl=/opt/mp --enable-lto --enable-plugin
--with-arch=corei7 --with-cpu=corei7
(tested with
make -k check-gfortran RUNTESTFLAGS="vect.exp=fast-math-mgrid-resid.f
--target_board=unix'{-m32,-m64}'").
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
` (5 preceding siblings ...)
2014-01-13 12:36 ` dominiq at lps dot ens.fr
@ 2014-01-14 9:01 ` jakub at gcc dot gnu.org
2014-01-14 9:04 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-01-14 9:01 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Tue Jan 14 09:00:30 2014
New Revision: 206598
URL: http://gcc.gnu.org/viewcvs?rev=206598&root=gcc&view=rev
Log:
PR testsuite/59494
* gfortran.dg/vect/fast-math-mgrid-resid.f: Change
-fdump-tree-optimized to -fdump-tree-pcom-details in dg-options and
cleanup-tree-dump from optimized to pcom. Remove scan-tree-dump-times
for vect_\[^\\n\]*\\+, add scan-tree-dump-times for no suitable chains and
Executing predictive commoning without unrolling.
Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gfortran.dg/vect/fast-math-mgrid-resid.f
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug testsuite/59494] [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13
2013-12-13 13:33 [Bug testsuite/59494] New: [4.9 Regression] FAIL: gfortran.dg/vect/fast-math-mgrid-resid.f scan-tree-dump-times optimized "vect_[^\\n]*\\+ " 13 dominiq at lps dot ens.fr
` (6 preceding siblings ...)
2014-01-14 9:01 ` jakub at gcc dot gnu.org
@ 2014-01-14 9:04 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-01-14 9:04 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59494
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #10 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.
^ permalink raw reply [flat|nested] 9+ messages in thread