http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #28 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-03-07
03:08:12 UTC ---
Author: jvdelisle
Date: Mon Mar 7 03:08:08 2011
New Revision: 170732
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170732
Log:
2011-03-06 Jakub
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #29 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-03-07
03:14:03 UTC ---
Author: jvdelisle
Date: Mon Mar 7 03:14:00 2011
New Revision: 170733
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170733
Log:
2011-03-06 Jerry
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #27 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-03-05
00:05:40 UTC ---
Author: jvdelisle
Date: Sat Mar 5 00:05:34 2011
New Revision: 170692
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170692
Log:
2011-03-04 Jakub
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
Richard Guenther rguenth at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
Richard Guenther rguenth at gcc dot gnu.org changed:
What|Removed |Added
Target|x86_64-*-* |x86_64-*-*,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
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=47878
--- Comment #3 from Tobias Burnus burnus at gcc dot gnu.org 2011-02-24
10:57:28 UTC ---
(In reply to comment #0)
I see miscompares with -O3 and -O2 -ffast-math.
Does the miscompare depend on the -O flags? If so, the patch for PR 47694
cannot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #4 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
11:19:57 UTC ---
(In reply to comment #3)
(In reply to comment #0)
I see miscompares with -O3 and -O2 -ffast-math.
Does the miscompare depend on the -O flags? If so,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #5 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
11:28:36 UTC ---
Fails with -O0, works with r170432 reverted.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #6 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
13:51:09 UTC ---
We are going to need some sort of reduced test case or revert the patch.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #7 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
14:01:27 UTC ---
The error is
Performing comparison with probe entries
1 ar2:global move determines position 58 45, similarity = 0.79937
1 ar2: best
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #8 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
14:04:04 UTC ---
Ah, line 61 of that file is
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #9 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
14:18:06 UTC ---
And
Integer, Pointer:: Image (:, :)
Breakpoint 1, read_radix (dtp=0x7fffd7e0, f=0x107bba0, dest=0x6377c0 �,
length=4,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #10 from Richard Guenther rguenth at gcc dot gnu.org 2011-02-24
14:25:48 UTC ---
There are a load of valgrind errors as well:
==11765== Invalid read of size 1
==11765==at 0x4F003D6: eat_leading_spaces (read.c:499)
==11765==by
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #11 from Tobias Burnus burnus at gcc dot gnu.org 2011-02-24
14:34:09 UTC ---
Failing test case - though different to the one reported above:
integer :: a(5)
open(99,Recl = 40)
write(99,'(5i3)') 1,2,3
rewind(99)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #12 from Tobias Burnus burnus at gcc dot gnu.org 2011-02-24
14:35:54 UTC ---
(In reply to comment #11)
Test case for the actual issue:
The crucial part seems to be to have (a) RECL= and (b) a line which is longer
than 80
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #13 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-24
15:05:39 UTC ---
Created attachment 23455
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=23455
gcc46-pr47878.patch
So how about this patch? Fixes the valgrind reported
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #15 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
16:03:41 UTC ---
Jakub's suggestion is probably OK. I am not at a place where I can do and
test.
My only concern, is there a chance that we would backup passed the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #16 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-24
16:20:27 UTC ---
Do you prefer the patch in #c13, or what I wrote in #c14? I'd prefer not to
test both.
As for pointer comparisons, you can't compare the old pointer with the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #17 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
17:07:06 UTC ---
C14, my browser here is not displaying the comment numbers, odd.
You will see where we set the eor number to 1 or 2 depending on what was found.
That can
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #18 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
17:10:50 UTC ---
The old code did this:
fbuf_seek (dtp-u.p.current_unit, n + seen_comma, SEEK_CUR);
So we could adjust for comma as well.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #20 from Tobias Burnus burnus at gcc dot gnu.org 2011-02-24
17:41:20 UTC ---
Comment on attachment 23456
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=23456
gcc46-pr47878.patch
+ read (99, '(5i3)') a
+ if (any (a.ne.(/1, 2, 3,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #21 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
18:27:42 UTC ---
This is OK and do add the close with status = delete
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #22 from Jerry DeLisle jvdelisle at gcc dot gnu.org 2011-02-24
18:30:38 UTC ---
Just to be clear;
Created attachment 23456 [details]
gcc46-pr47878.patch
is approved. It has to be right, look at that attachment number!
;)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #23 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-24
20:14:00 UTC ---
Author: jakub
Date: Thu Feb 24 20:13:57 2011
New Revision: 170476
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170476
Log:
PR fortran/47878
*
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
--- Comment #24 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-24
20:17:05 UTC ---
Richard, can you check if this fixes 187.facerec?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47878
Andrew Pinski pinskia at gcc dot gnu.org changed:
What|Removed |Added
CC||hjl.tools at
29 matches
Mail list logo
