[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-14 Thread patchapp at dberlin dot org
--- Comment #11 from patchapp at dberlin dot org 2007-06-14 12:00 --- Subject: Bug number PR32302 A patch for this bug has been added to the patch tracker. The mailing list url for the patch is http://gcc.gnu.org/ml/gcc-patches/2007-06/msg00954.html --

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-14 Thread pault at gcc dot gnu dot org
--- Comment #12 from pault at gcc dot gnu dot org 2007-06-14 13:04 --- Subject: Bug 32302 Author: pault Date: Thu Jun 14 13:04:05 2007 New Revision: 125708 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=125708 Log: 2007-06-14 Paul Thomas [EMAIL PROTECTED] PR

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread pault at gcc dot gnu dot org
--- Comment #5 from pault at gcc dot gnu dot org 2007-06-13 09:01 --- Dear All, I think that you are probably right - it looks like my patch for PR29786 is to blame, since it is the only thing to touch trans-common.c for a long time. This is incorrect - it goes back at least as

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread pault at gcc dot gnu dot org
--- Comment #6 from pault at gcc dot gnu dot org 2007-06-13 09:04 --- PS - It also explains a failure by gfortran to produce a working version of the PROTEUS tokamak equilibrium code. I just tried, for the first time, to compile it with no optimization. It is SLOW but it goes. It

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread pault at gcc dot gnu dot org
--- Comment #7 from pault at gcc dot gnu dot org 2007-06-13 09:30 --- The problem lies with the common block aux32, which is declared in subroutine unpki with 6 integer(4) arrays at the beginning and, elsewhere, is declared with default reals. Apparently, this makes a mess of the

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread pault at gcc dot gnu dot org
--- Comment #8 from pault at gcc dot gnu dot org 2007-06-13 12:26 --- This fixes it but is, as yet unregtested. I'll do the business tonight and will commit as 'obvious', if all is well. Thanks, Dale! Paul Index: gcc/fortran/trans-common.c

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread dir at lanl dot gov
--- Comment #9 from dir at lanl dot gov 2007-06-13 13:39 --- Your Welcome Paul, With a little luck, the fix for this will cure the remaining problems that I have been having with my programs when optimization is turned on. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32302

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-13 Thread pault at gcc dot gnu dot org
--- Comment #10 from pault at gcc dot gnu dot org 2007-06-13 15:00 --- I goofed - the previous does not fix it at all. I hope that I am not too late to stop folk from building with this patch. Cheers Paul PS Compiling the subroutines separately or padding out the first aux32 both

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-12 Thread pinskia at gcc dot gnu dot org
--- Comment #2 from pinskia at gcc dot gnu dot org 2007-06-12 15:09 --- Until I know for sure, i am moving this back to the fortran component, it might be a front end issuse still. -- pinskia at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-12 Thread pault at gcc dot gnu dot org
--- Comment #3 from pault at gcc dot gnu dot org 2007-06-13 04:58 --- (In reply to comment #2) Until I know for sure, i am moving this back to the fortran component, it might be a front end issuse still. Andrew, I think that you are probably right - it looks like my patch for

[Bug fortran/32302] [4.2/4.3 Regression] Incorrect result with -O2

2007-06-12 Thread kargl at gcc dot gnu dot org
--- Comment #4 from kargl at gcc dot gnu dot org 2007-06-13 05:09 --- (In reply to comment #3) (In reply to comment #2) Until I know for sure, i am moving this back to the fortran component, it might be a front end issuse still. Andrew, I think that you are probably right