http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #11 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-13 12:31:03 UTC ---
(In reply to comment #10)
Draft patch (replaces the one in comment 9):
--- a/gcc/fortran/resolve.c
+++ b/gcc/fortran/resolve.c
@@
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #12 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-13
14:57:45 UTC ---
Author: burnus
Date: Thu Sep 13 14:57:38 2012
New Revision: 191259
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=191259
Log:
2012-09-13 Tobias Burnus
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #13 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-13
18:10:13 UTC ---
Author: burnus
Date: Thu Sep 13 18:10:04 2012
New Revision: 191275
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=191275
Log:
2012-09-13 Tobias Burnus
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #14 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-13
18:16:01 UTC ---
Author: burnus
Date: Thu Sep 13 18:15:52 2012
New Revision: 191276
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=191276
Log:
2012-09-13 Tobias Burnus
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
Tobias Burnus burnus at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #16 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-14 05:57:51 UTC ---
(In reply to comment #15)
FIXED on the trunk - and on the 4.6/4.7 branch. Sorry for the breakage!
Thank you and other gcc experts for
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
Tobias Burnus burnus at gcc dot gnu.org changed:
What|Removed |Added
CC||burnus at gcc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #1 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 11:41:12 UTC ---
the two revisions lead to a lot of changes, all these files differ in their
disassembled form:
1admm_methods.o Files f1 and f2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #2 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 20:11:24 UTC ---
some progress.. the object file that leads to wrong results is
parallel_rng_types.o. I'll see if I can get some further insight.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #3 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-12
20:22:58 UTC ---
(In reply to comment #2)
some progress.. the object file that leads to wrong results is
parallel_rng_types.o. I'll see if I can get some further insight.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #4 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 20:26:49 UTC ---
(In reply to comment #3)
(In reply to comment #2)
some progress.. the object file that leads to wrong results is
parallel_rng_types.o.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #5 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 20:46:05 UTC ---
Created attachment 28179
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=28179
testcase
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #6 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 20:50:40 UTC ---
The testcase illustrates the issue, compiling as
gfortran -c -O1 test.f90 -fdump-tree-optimized
shows that rn32 is only called once from
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #7 from Joost VandeVondele Joost.VandeVondele at mat dot ethz.ch
2012-09-12 20:58:23 UTC ---
(In reply to comment #6)
So I guess rn32 is incorrectly marked as pure.
which indeed is also visible in the .mod file:
'rn32'
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #8 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-12
21:00:32 UTC ---
Test case: Compile in two files with -O0/-O1/-O2/ give the expected i == 5 but
using -O3 hoists the i = s(x) out of the loop and thus gives i == 1.
Question:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
--- Comment #9 from Tobias Burnus burnus at gcc dot gnu.org 2012-09-12
21:11:54 UTC ---
Untested patch.
The first and second part allows VALUE for implicit_pure (in line with F2008
for PURE).
The third part is the crucial change: If there is a
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54556
Tobias Burnus burnus at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last
17 matches
Mail list logo