[Bug libfortran/45723] opening /dev/null for appending writes
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-09-22 15:07 --- I don't see a need to backport. Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45723
[Bug fortran/45748] -fimplicit-none strange behaviour
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-09-22 19:09 --- Please post result of gfortran -v -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45748
[Bug fortran/45748] -fimplicit-none strange behaviour
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-09-22 19:12 --- I do not see the problem with gfortran 4.3.4 on Cygwin -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45748
[Bug fortran/45710] Adjust format and padding for WRITE of NAMELIST group to internal file
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-09-23 00:53 --- Subject: Bug 45710 Author: jvdelisle Date: Thu Sep 23 00:53:15 2010 New Revision: 164543 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164543 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45710 * io/write.c (namelist_write_newline): Pad character array internal unit records with spaces. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/write.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug fortran/45532] gfortran namelist read error
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2010-09-23 00:58 --- Subject: Bug 45532 Author: jvdelisle Date: Thu Sep 23 00:58:12 2010 New Revision: 164544 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164544 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45532 Backport from mainline: * io/list_read.c (nml_get_obj_data): Set first_nl if the previous is NULL. Modified: branches/gcc-4_5-branch/libgfortran/ChangeLog branches/gcc-4_5-branch/libgfortran/io/list_read.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:11 --- Subject: Bug 45532 Author: jvdelisle Date: Thu Sep 23 01:11:16 2010 New Revision: 164545 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164545 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org Backport from mainline PR libfortran/45532 * gfortran.dg/namelist_64.f90: New test. Added: branches/gcc-4_5-branch/gcc/testsuite/gfortran.dg/namelist_64.f90 Modified: branches/gcc-4_5-branch/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:19 --- Subject: Bug 45532 Author: jvdelisle Date: Thu Sep 23 01:19:13 2010 New Revision: 164546 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164546 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org Backport from mainline: PR libfortran/45532 * io/list_read.c (nml_get_obj_data): Set first_nl if the previous is NULL. Modified: branches/gcc-4_4-branch/libgfortran/ChangeLog branches/gcc-4_4-branch/libgfortran/io/list_read.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:21 --- Subject: Bug 45532 Author: jvdelisle Date: Thu Sep 23 01:21:10 2010 New Revision: 164547 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164547 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org Backport from mainline PR libfortran/45532 * gfortran.dg/namelist_64.f90: New test. Added: branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/namelist_64.f90 Modified: branches/gcc-4_4-branch/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:22 --- Backport complete. Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45710] Adjust format and padding for WRITE of NAMELIST group to internal file
--- Comment #13 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:28 --- Subject: Bug 45710 Author: jvdelisle Date: Thu Sep 23 01:28:39 2010 New Revision: 164548 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164548 Log: 2010-09-22 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45710 * gfortran.dg/namelist_65.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/namelist_65.f90 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug fortran/45710] Adjust format and padding for WRITE of NAMELIST group to internal file
--- Comment #14 from jvdelisle at gcc dot gnu dot org 2010-09-23 01:43 --- Fixed on trunk. I will backport to 4.5 and 4.4 except the portion dealing with char4 internal units which is not supported on these earlier releases. Backport will occur in a few days or weeks. I would like to give some burnin time on trunk. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug fortran/45710] Adjust format and padding for WRITE of NAMELIST group to internal file
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-09-22 02:26 --- Changed summary of this bug to reflect that I will be adjusting the padding of internal unit character arrays and initial space on some lines. On my system I see: NL1 JUNK space A= 1.000,JUNK space B= 2.000,JUNK space C= 3.000,JUNK space / I am not sure on the no space on the first line. I will pad out the strings with blanks. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Summary|WRITE of NAMELIST group to |Adjust format and padding |internal file contains |for WRITE of NAMELIST group |newline characters |to internal file http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug fortran/45710] Adjust format and padding for WRITE of NAMELIST group to internal file
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-09-22 02:52 --- Created an attachment (id=21860) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21860action=view) Tentative patch, not regression tested Our interpretation of this: 3 An ampersand character followed immediately by a namelist-group-name will be produced by namelist formatting at the start of the first output record to indicate which particular group of data objects is being output. A slash is produced by namelist formatting to indicate the end of the namelist formatting. Is no space on the first line. I would like to leave it that way for now. Attached patch fixes the padding. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug libfortran/45723] opening /dev/null for appending writes
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-09-22 03:10 --- Subject: Bug 45723 Author: jvdelisle Date: Wed Sep 22 03:10:24 2010 New Revision: 164507 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164507 Log: 2010-09-21 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45723 * io/open.c (new_unit): On POSITION_APPEND don't seek if file length is zero. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/open.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45723
[Bug fortran/45715] [ABI cleanup] Move runtime parsing of I/O control list to front end
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-09-21 03:08 --- Here is a case where we do have to evaluate at run time. So maybe we can not do what is proposed above. character(10):: advancevar advancevar = no write(*, (a), advance=advancevar) Well, print *, Does it work? end ! Interestingly, if advancevar is a PARAMETER, we get an error. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45715
[Bug fortran/45710] WRITE of NAMELIST group to internal file contains newline characters
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2010-09-21 03:20 --- With current trunk, gfortran 4.6, I get: BEFORE: 001 @@@ 002 @@@ 003 @@@ 004 @@@ 005 @@@ AFTER: NL1 @@@ A= 1.000,@ B= 2.000,@ C= 3.000,@ / I thought I remembered fixing this before. What version of gfortran are you using? (I snipped off the ends of the lines above to fit the report.) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug libfortran/45723] opening /dev/null for appending writes
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-09-19 20:01 --- This is an OS specific error. libgfortran is attempting to seek to the end of the file to prepare for appending. Obviously, one can not seek /dev/null. I will see if we can detect that the file is unseekable and if that is the case we can start appending AS-IS. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-09-19 20:01:18 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45723
[Bug libfortran/45723] opening /dev/null for appending writes
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-09-20 04:46 --- Created an attachment (id=21841) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21841action=view) Possible patch This patch passes regression testing. Don't seek if filesize is zero. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45723
[Bug fortran/45710] WRITE of NAMELIST group to internal file contains newline characters
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-09-18 18:18 --- This issue, line ending for namelist to character arrays, has come up before. I don't remember the outcome of that previous discussion. I will research a little. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-09-18 18:18:59 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45710
[Bug fortran/45532] gfortran namelist read error
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-09-14 04:37 --- Subject: Bug 45532 Author: jvdelisle Date: Tue Sep 14 04:37:02 2010 New Revision: 164266 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164266 Log: 2010-09-14 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45532 * io/list_read.c (nml_get_obj_data): Set first_nl if the previous is NULL. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/list_read.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2010-09-14 04:39 --- Subject: Bug 45532 Author: jvdelisle Date: Tue Sep 14 04:39:13 2010 New Revision: 164267 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164267 Log: 2010-09-14 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45532 * gfortran.dg/namelist_64.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/namelist_64.f90 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45532] gfortran namelist read error
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-09-14 04:40 --- Fixed on trunk, will backport to 4.4 and 4.5 in a few days. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Target Milestone|--- |4.6.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/29602] [F2003] I/O specifiers can now be of any kind
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-09-12 15:38 --- When we break abi for kind=8 recl, we should fix all of these. I will assign to myself so I don't forget. Moving out of enhancement status. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Severity|enhancement |normal Status|NEW |ASSIGNED Last reconfirmed|2006-10-26 08:31:50 |2010-09-12 15:38:56 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29602
[Bug fortran/30123] Document INQUIRE, especially UNFORMATTED and FORMATTED
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-09-12 15:42 --- Adding to my list -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|NEW |ASSIGNED Last reconfirmed|2007-02-16 16:19:38 |2010-09-12 15:42:44 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30123
[Bug fortran/31139] sum(w_re(1:nn,1)*fi(i(1:nn, ii))) up to 3.5x slower than C version
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2010-09-12 15:45 --- I have not tested this with latest trunk, but I wonder if any of the recent optimization work has improved this. Can it be closed yet? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31139
[Bug fortran/36841] Eliminate gfortran_sum_r8 call for calculation involving multidimensional array multiplication followed by a sum along first dimension
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-09-12 16:14 --- Is this something for the FE to do? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36841
[Bug fortran/45532] gfortran namelist read error
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-09-08 19:03 --- This is very interesting (aka a real bugger): Reversing the order of the declarations in type curve_struct type (line_struct) line type (symbol_struct) symbol end type to type curve_struct type (symbol_struct) symbol type (line_struct) line end type allows the read to proceed correctly. Stay tuned. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45608] New: Bogus error about procedure attribute
This test case is invalid. It is rejected with an error message that is nonsense in this context. program test implicit none type line_struct integer :: width = 10 end type type symbol_struct integer :: typee = 10 end type type curve_struct type (line_struct) line type (symbol_struct) symbol end type type (curve_struct) curve namelist / params / curve ! open (1, file = 'test.nml') read (1, nml = params) print *, curve(1)%symbol%typee end program $ gfc -g test.f90 test.f90:20.17: namelist / params / curve 1 Error: PROCEDURE attribute conflicts with NAMELIST attribute in 'curve' at (1) Another example is: program test2 implicit none type line_struct integer :: width = 10 end type type symbol_struct integer :: typee = 10 end type type curve_struct type (line_struct) line type (symbol_struct) symbol end type type (curve_struct) curve print *, curve(1)%symbol%typee end program $ gfc -g test2.f90 /tmp/ccuBUha0.o: In function `test': /home/jerry/prs/pr45532/pr45532.f90:20: undefined reference to `curve_' collect2: ld returned 1 exit status -- Summary: Bogus error about procedure attribute Product: gcc Version: 4.6.0 Status: UNCONFIRMED Keywords: diagnostic Severity: normal Priority: P3 Component: fortran AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: jvdelisle at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45608
[Bug fortran/45532] gfortran namelist read error
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-09-05 19:27 --- I will have a go at this. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|NEW |ASSIGNED Last reconfirmed|2010-09-04 07:35:23 |2010-09-05 19:27:55 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45532
[Bug fortran/45463] gfortran internal write bug
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-09-01 02:10 --- Update: More responses in comp.lang.fortran bring up the point that if trim(line) is supposed to return a temporary, the code might be conforming. Seems that the situation is not clear... Where in the standard does it say that trim(line) must generate a temporary. I can think of an implementation that would track the length trimmed to and conserve the memory. My point being that this level of behavior, using temporaries or not, IMNSHO, is implementation detail that does not belong in the standard if it is there. c.l.f folks sometimes get a bit esoteric in their discussions. I am closing this bug as not a bug (invalid). I do appreciate the report though. We do not want to miss things and its better to error on the safe side and report then not. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||INVALID http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45463
[Bug fortran/43217] Output of Hollerith constants which are not a multiple of 4 bytes
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-08-27 06:50 --- Subject: Bug 43217 Author: jvdelisle Date: Fri Aug 27 06:50:03 2010 New Revision: 163581 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163581 Log: 2010-08-27 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/43217 * primary.c (match_hollerith_constant): Calculate padding needed to fill default integer and allocate string for that size. Set pad bytes to ' '. * gfortran.h: Add hollerith pad value to type spec union. * data.c (create_character_initializer): Fix spelling of function name. Use hollerith pad value to calculate length. * arith.c (hollerith2representation); Use hollerith pad value to calculate length. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/arith.c trunk/gcc/fortran/data.c trunk/gcc/fortran/gfortran.h trunk/gcc/fortran/primary.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43217
[Bug fortran/43217] Output of Hollerith constants which are not a multiple of 4 bytes
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-08-27 14:02 --- Dominiq, can you test the output of the test case for me on PPC? hollerith8.f90 in the gfortran post. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43217
[Bug fortran/43217] Output of Hollerith constants which are not a multiple of 4 bytes
--- Comment #14 from jvdelisle at gcc dot gnu dot org 2010-08-27 15:00 --- Subject: Bug 43217 Author: jvdelisle Date: Fri Aug 27 15:00:11 2010 New Revision: 163588 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163588 Log: 2010-08-27 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/43217 * gfortran.dg/hollerith8.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/hollerith8.f90 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43217
[Bug fortran/43217] Output of Hollerith constants which are not a multiple of 4 bytes
--- Comment #15 from jvdelisle at gcc dot gnu dot org 2010-08-27 15:01 --- Dominique, thanks for testing that. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43217
[Bug fortran/41859] ICE on invalid expression involving DT with pointer components in I/O
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-08-26 02:13 --- Subject: Bug 41859 Author: jvdelisle Date: Thu Aug 26 02:12:49 2010 New Revision: 163561 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163561 Log: 2010-08-25 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/41859 * gfortran.dg/der_io_4.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/der_io_4.f90 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41859
[Bug fortran/43217] Output of Hollerith constants which are not a multiple of 4 bytes
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2010-08-21 16:42 --- Created an attachment (id=21539) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21539action=view) Proposed patch This patch adds the padding requested. Output of test case matches a few of the other compilers. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43217
[Bug fortran/45108] Namelist read: Not aborted when reading from STDIN
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-08-19 20:28 --- Subject: Bug 45108 Author: jvdelisle Date: Thu Aug 19 20:28:09 2010 New Revision: 163387 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163387 Log: 2010-08-19 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45108 * io/list_read.c (namelist_read): If namelist reading fails, use generate_error and then continue the read loop. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/list_read.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45108
[Bug fortran/45108] Namelist read: Not aborted when reading from STDIN
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-08-19 20:32 --- Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45108
[Bug fortran/45143] [F2008,corrig1] Endless loop with unlimited edit descriptor
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2010-08-19 22:56 --- This is not currently diagnosed during compilation. My original plan is to see if I can do that with the front end and then close it. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45143
[Bug fortran/41859] ICE on invalid expression involving DT with pointer components in I/O
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-08-19 02:36 --- Subject: Bug 41859 Author: jvdelisle Date: Thu Aug 19 02:35:45 2010 New Revision: 163363 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163363 Log: 2010-08-19 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/41859 * resolve.c (resolve_transfer): Traverse operands and set expression to be checked to a non EXPR_OP type. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/resolve.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41859
[Bug fortran/45108] Namelist read: Not aborted when reading from STDIN
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-08-19 03:52 --- Created an attachment (id=21516) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21516action=view) Proposed patch This patch will generate an error. If IOSTAT or IOMSG is used, the read will continue in the loop after an error, but the error status variables will be set so that one will know an error occurred when the READ has completed. Any thoughts Tobias? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45108
[Bug fortran/41859] ICE on invalid expression involving DT with pointer components in I/O
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-08-19 04:01 --- Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41859
[Bug fortran/41859] ICE on invalid expression involving DT with pointer components in I/O
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-08-16 06:09 --- The problem resides in resolve_transfer. The checks are not performed because the expression is wrapped in parenthesis and so the expression is of type EXPR_OP, so resolve_transfer is just exited. The following patch simply finds the first operand that is not of type EXPR_OP and sets exp to that for checking. Index: resolve.c === --- resolve.c (revision 163259) +++ resolve.c (working copy) @@ -7696,6 +7696,12 @@ exp = code-expr1; + while (exp != NULL exp-expr_type == EXPR_OP) +exp = exp-value.op.op1; + + if (exp == NULL) +return; + if (exp-expr_type != EXPR_VARIABLE exp-expr_type != EXPR_FUNCTION) return; -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41859
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #13 from jvdelisle at gcc dot gnu dot org 2010-08-17 03:13 --- Closing now -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/45108] Namelist read: Not aborted when reading from STDIN
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-08-17 03:18 --- There was some history regarding legacy codes as the reason why this does not error out. ie looping back and trying again was a feature. I think I may put this behind -std=legacy. It does seem odd the way we are doing this now. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45108
[Bug bootstrap/45285] New: Bootstrap failure on Cygwin
Bootstrap comparison failure! i686-pc-cygwin/libgomp/.libs/bar.o differs i686-pc-cygwin/libgomp/.libs/barrier.o differs i686-pc-cygwin/libgomp/.libs/critical.o differs i686-pc-cygwin/libgomp/.libs/env.o differs i686-pc-cygwin/libgomp/.libs/iter.o differs i686-pc-cygwin/libgomp/.libs/iter_ull.o differs i686-pc-cygwin/libgomp/.libs/lock.o differs i686-pc-cygwin/libgomp/.libs/loop.o differs i686-pc-cygwin/libgomp/.libs/loop_ull.o differs i686-pc-cygwin/libgomp/.libs/ordered.o differs i686-pc-cygwin/libgomp/.libs/parallel.o differs i686-pc-cygwin/libgomp/.libs/proc.o differs i686-pc-cygwin/libgomp/.libs/sections.o differs i686-pc-cygwin/libgomp/.libs/single.o differs i686-pc-cygwin/libgomp/.libs/task.o differs i686-pc-cygwin/libgomp/.libs/team.o differs i686-pc-cygwin/libgomp/.libs/work.o differs i686-pc-cygwin/libgomp/bar.o differs i686-pc-cygwin/libgomp/barrier.o differs i686-pc-cygwin/libgomp/critical.o differs i686-pc-cygwin/libgomp/env.o differs i686-pc-cygwin/libgomp/iter.o differs i686-pc-cygwin/libgomp/iter_ull.o differs i686-pc-cygwin/libgomp/lock.o differs i686-pc-cygwin/libgomp/loop.o differs i686-pc-cygwin/libgomp/loop_ull.o differs i686-pc-cygwin/libgomp/ordered.o differs i686-pc-cygwin/libgomp/parallel.o differs i686-pc-cygwin/libgomp/proc.o differs i686-pc-cygwin/libgomp/sections.o differs i686-pc-cygwin/libgomp/single.o differs i686-pc-cygwin/libgomp/task.o differs i686-pc-cygwin/libgomp/team.o differs i686-pc-cygwin/libgomp/work.o differs make[2]: *** [compare] Error 1 make[2]: Leaving directory `/home/Jerry/gcc/objdir' make[1]: *** [stage3-bubble] Error 2 make[1]: Leaving directory `/home/Jerry/gcc/objdir' make: *** [all] Error 2 I am trying a non-bootstrap build now to see if I can get something working, then I will go back to bootstrap attempt. Stay tuned -- Summary: Bootstrap failure on Cygwin Product: gcc Version: 4.6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: bootstrap AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: jvdelisle at gcc dot gnu dot org GCC host triplet: Cygwin http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45285
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-08-14 18:59 --- Subject: Bug 44931 Author: jvdelisle Date: Sat Aug 14 18:59:18 2010 New Revision: 163245 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163245 Log: 2010-08-14 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/44931 * io/inquire.c (inquire_via_unit): Add special case for __MINGW32__ to return special file names CONIN$, CONOUT$, and CONERR$. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/inquire.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/45186] [4.5/4.6 Regression] Gfortran 4.5.0 emits wrong linenumbers
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2010-08-13 15:11 --- I will try the other patch and see what this does. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45186
[Bug fortran/45186] [4.5/4.6 Regression] Gfortran 4.5.0 emits wrong linenumbers
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-08-13 17:24 --- With or without the other patch, the gimple code has: main (integer(kind=4) argc, character(kind=1) * * argv) [tx_f90_pointers.f90 : 30:0] { integer(kind=4) D.1535; static integer(kind=4) options.0[8] = {68, 511, 0, 0, 0, 1, 0, 1}; [tx_f90_pointers.f90 : 30:0] _gfortran_set_args (argc, argv); [tx_f90_pointers.f90 : 30:0] _gfortran_set_options (8, [tx_f90_pointers.f90 : 30] options.0[0]); [tx_f90_pointers.f90 : 30:0] test (); [tx_f90_pointers.f90 : 30:0] D.1535 = 0; [tx_f90_pointers.f90 : 30:0] return D.1535; } test () [tx_f90_pointers.f90 : 3:0] { integer(kind=4) D.1540; integer(kind=4) array1[10]; integer(kind=4) array2[100]; integer(kind=4) i; struct array1_integer(kind=4) p1; struct array1_integer(kind=4) p11; [tx_f90_pointers.f90 : 3:0] p1.data = 0B; [tx_f90_pointers.f90 : 3:0] p11.data = 0B; [tx_f90_pointers.f90 : 12:0] p1.data = 0B; [tx_f90_pointers.f90 : 3:0] i = 1; [tx_f90_pointers.f90 : 3:0] if (i = 10) goto D.1537; else goto D.1538; D.1537: D.1539: [tx_f90_pointers.f90 : 3:0] { logical(kind=4) D.1529; [tx_f90_pointers.f90 : 15:0] D.1540 = i + -1; [tx_f90_pointers.f90 : 15:0] array1[D.1540] = i; L.1: [tx_f90_pointers.f90 : 3:0] D.1529 = i == 10; [tx_f90_pointers.f90 : 3:0] i = i + 1; [tx_f90_pointers.f90 : 3:0] if (D.1529 != 0) goto L.2; else goto D.1541; D.1541: } [tx_f90_pointers.f90 : 3:0] goto D.1539; D.1538: L.2: [tx_f90_pointers.f90 : 3:0] p1.dtype = 265; [tx_f90_pointers.f90 : 3:0] p1.dim[0].lbound = 1; [tx_f90_pointers.f90 : 3:0] p1.dim[0].ubound = 5; [tx_f90_pointers.f90 : 3:0] p1.dim[0].stride = -2; [tx_f90_pointers.f90 : 3:0] p1.data = array1[8]; [tx_f90_pointers.f90 : 3:0] p1.offset = 2; [tx_f90_pointers.f90 : 3:0] p11.dtype = 265; [tx_f90_pointers.f90 : 3:0] p11.dim[0].lbound = 1; [tx_f90_pointers.f90 : 3:0] p11.dim[0].ubound = 5; [tx_f90_pointers.f90 : 3:0] p11.dim[0].stride = 2; [tx_f90_pointers.f90 : 3:0] p11.data = array1[0]; [tx_f90_pointers.f90 : 3:0] p11.offset = -2; [tx_f90_pointers.f90 : 23:0] i = 1; [tx_f90_pointers.f90 : 3:0] p1.dtype = 265; [tx_f90_pointers.f90 : 3:0] p1.dim[0].lbound = 1; [tx_f90_pointers.f90 : 3:0] p1.dim[0].ubound = 10; [tx_f90_pointers.f90 : 3:0] p1.dim[0].stride = 1; [tx_f90_pointers.f90 : 3:0] p1.data = array2[0]; [tx_f90_pointers.f90 : 3:0] p1.offset = -1; [tx_f90_pointers.f90 : 28:0] p11 = p1; } The entire DO LOOP is one line. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45186
[Bug fortran/42526] bogus truncation warning for default-initialized character components
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-08-13 05:14 --- Subject: Bug 42526 Author: jvdelisle Date: Fri Aug 13 05:14:18 2010 New Revision: 163220 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=163220 Log: 2010-08-12 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/42526 * gfortran.dg/char_component_initializer_1.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/char_component_initializer_2.f90 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42526
[Bug fortran/42526] bogus truncation warning for default-initialized character components
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-08-13 05:24 --- Closing. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42526
[Bug fortran/45128] Segmentation fault with -fwhole-file for subref_array_pointer
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-08-09 17:18 --- Paul, can you send me a preview of the new descriptor structure so I can be planning the internal unit I/O impacts if any. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45128
[Bug fortran/44998] runtime error when reading complex*16 using formatted I/O [reopen bug 19064]
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-08-09 00:31 --- Good -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44998
[Bug fortran/45143] [F2008,corrig1] Endless loop with unlimited edit descriptor
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-08-07 12:03 --- Subject: Bug 45143 Author: jvdelisle Date: Sat Aug 7 12:03:23 2010 New Revision: 162978 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162978 Log: 2010-08-07 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45143 * io/format.c: Remove fnode storage structure definitions, moving these to format.h. (parse_format_list): Add check for data descriptors, taking care of nested formats. Adjust calling parameters to pass a check flag. (parse_format): Likewise. * io/format.h: Add structures moved from format.c. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/format.c trunk/libgfortran/io/format.h -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45143
[Bug fortran/45143] [F2008,corrig1] Endless loop with unlimited edit descriptor
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-08-07 12:10 --- Subject: Bug 45143 Author: jvdelisle Date: Sat Aug 7 12:10:25 2010 New Revision: 162979 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162979 Log: 2010-08-07 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/45143 * gfortran.dg/fmt_error_11.f03: New test. Added: trunk/gcc/testsuite/gfortran.dg/fmt_error_11.f03 Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45143
[Bug fortran/43954] [4.3/4.4 regression] gfortran does not support -Wp, -MD for *.F
--- Comment #15 from jvdelisle at gcc dot gnu dot org 2010-08-07 15:43 --- I have the patch for backport to 4.5 applied to my local 4.5 branch and I am regression testing now. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] [4.3/4.4 regression] gfortran does not support -Wp, -MD for *.F
--- Comment #16 from jvdelisle at gcc dot gnu dot org 2010-08-07 16:52 --- Subject: Bug 43954 Author: jvdelisle Date: Sat Aug 7 16:51:55 2010 New Revision: 162980 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162980 Log: 2010-08-07 Daniel Franke franke.dan...@gmail.com 2010-06-13 Daniel Franke franke.dan...@gmail.com PR fortran/31588 PR fortran/43954 Backport from mainline: * gfortranspec.c (lang_specific_driver): Removed deprecation warning for -M. * lang.opt: Add options -M, -MM, -MD, -MMD, -MF, -MG, -MP, -MT, -MQ. * lang-specs.h (CPP_FORWARD_OPTIONS): Add -M* options. * cpp.h (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. * cpp.c (gfc_cpp_option): Add deps* members. (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. (gfc_cpp_init_options): Initialize new options. (gfc_cpp_handle_option): Handle new options. (gfc_cpp_post_options): Map new options to libcpp-options. (gfc_cpp_init): Handle deferred -MQ and -MT options. (gfc_cpp_done): If requested, write dependencies to file. * module.c (gfc_dump_module): Add a module filename as target. * scanner.c (open_included_file): New parameter system; add the included file as dependency. (gfc_open_included_file): Add the included file as dependency. (gfc_open_intrinsic_module): Likewise. * invoke.texi: Removed deprecation warning for -M. * gfortran.texi: Removed Makefile-dependencies project. Modified: branches/gcc-4_5-branch/gcc/fortran/ChangeLog branches/gcc-4_5-branch/gcc/fortran/cpp.c branches/gcc-4_5-branch/gcc/fortran/cpp.h branches/gcc-4_5-branch/gcc/fortran/gfortran.texi branches/gcc-4_5-branch/gcc/fortran/gfortranspec.c branches/gcc-4_5-branch/gcc/fortran/invoke.texi branches/gcc-4_5-branch/gcc/fortran/lang-specs.h branches/gcc-4_5-branch/gcc/fortran/lang.opt branches/gcc-4_5-branch/gcc/fortran/module.c branches/gcc-4_5-branch/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/31588] gfortran should be able to output Makefile dependencies with -M* options
--- Comment #17 from jvdelisle at gcc dot gnu dot org 2010-08-07 16:52 --- Subject: Bug 31588 Author: jvdelisle Date: Sat Aug 7 16:51:55 2010 New Revision: 162980 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162980 Log: 2010-08-07 Daniel Franke franke.dan...@gmail.com 2010-06-13 Daniel Franke franke.dan...@gmail.com PR fortran/31588 PR fortran/43954 Backport from mainline: * gfortranspec.c (lang_specific_driver): Removed deprecation warning for -M. * lang.opt: Add options -M, -MM, -MD, -MMD, -MF, -MG, -MP, -MT, -MQ. * lang-specs.h (CPP_FORWARD_OPTIONS): Add -M* options. * cpp.h (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. * cpp.c (gfc_cpp_option): Add deps* members. (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. (gfc_cpp_init_options): Initialize new options. (gfc_cpp_handle_option): Handle new options. (gfc_cpp_post_options): Map new options to libcpp-options. (gfc_cpp_init): Handle deferred -MQ and -MT options. (gfc_cpp_done): If requested, write dependencies to file. * module.c (gfc_dump_module): Add a module filename as target. * scanner.c (open_included_file): New parameter system; add the included file as dependency. (gfc_open_included_file): Add the included file as dependency. (gfc_open_intrinsic_module): Likewise. * invoke.texi: Removed deprecation warning for -M. * gfortran.texi: Removed Makefile-dependencies project. Modified: branches/gcc-4_5-branch/gcc/fortran/ChangeLog branches/gcc-4_5-branch/gcc/fortran/cpp.c branches/gcc-4_5-branch/gcc/fortran/cpp.h branches/gcc-4_5-branch/gcc/fortran/gfortran.texi branches/gcc-4_5-branch/gcc/fortran/gfortranspec.c branches/gcc-4_5-branch/gcc/fortran/invoke.texi branches/gcc-4_5-branch/gcc/fortran/lang-specs.h branches/gcc-4_5-branch/gcc/fortran/lang.opt branches/gcc-4_5-branch/gcc/fortran/module.c branches/gcc-4_5-branch/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31588
[Bug fortran/42526] bogus truncation warning for default-initialized character components
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-08-07 21:10 --- Yes, fixed. I will make a test case out of this with the -Wall so we do not regress. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-08-07 21:10:59 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42526
[Bug fortran/31588] gfortran should be able to output Makefile dependencies with -M* options
--- Comment #18 from jvdelisle at gcc dot gnu dot org 2010-08-08 01:59 --- Subject: Bug 31588 Author: jvdelisle Date: Sun Aug 8 01:59:15 2010 New Revision: 162990 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162990 Log: 2010-08-07 Daniel Franke franke.dan...@gmail.com PR fortran/31588 PR fortran/43954 Backport from mainline: * gfortranspec.c (lang_specific_driver): Removed deprecation warning for -M. * lang.opt: Add options -M, -MM, -MD, -MMD, -MF, -MG, -MP, -MT, -MQ. * lang-specs.h (CPP_FORWARD_OPTIONS): Add -M* options. * cpp.h (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. * cpp.c (gfc_cpp_option): Add deps* members. (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. (gfc_cpp_init_options): Initialize new options. (gfc_cpp_handle_option): Handle new options. (gfc_cpp_post_options): Map new options to libcpp-options. (gfc_cpp_init): Handle deferred -MQ and -MT options. (gfc_cpp_done): If requested, write dependencies to file. * module.c (gfc_dump_module): Add a module filename as target. * scanner.c (open_included_file): New parameter system; add the included file as dependency. (gfc_open_included_file): Add the included file as dependency. (gfc_open_intrinsic_module): Likewise. * invoke.texi: Removed deprecation warning for -M. * gfortran.texi: Removed Makefile-dependencies project. Modified: branches/gcc-4_4-branch/gcc/fortran/ChangeLog branches/gcc-4_4-branch/gcc/fortran/cpp.c branches/gcc-4_4-branch/gcc/fortran/cpp.h branches/gcc-4_4-branch/gcc/fortran/gfortran.texi branches/gcc-4_4-branch/gcc/fortran/gfortranspec.c branches/gcc-4_4-branch/gcc/fortran/invoke.texi branches/gcc-4_4-branch/gcc/fortran/lang-specs.h branches/gcc-4_4-branch/gcc/fortran/lang.opt branches/gcc-4_4-branch/gcc/fortran/module.c branches/gcc-4_4-branch/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31588
[Bug fortran/43954] [4.4 regression] gfortran does not support -Wp, -MD for *.F
--- Comment #20 from jvdelisle at gcc dot gnu dot org 2010-08-08 01:59 --- Subject: Bug 43954 Author: jvdelisle Date: Sun Aug 8 01:59:15 2010 New Revision: 162990 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162990 Log: 2010-08-07 Daniel Franke franke.dan...@gmail.com PR fortran/31588 PR fortran/43954 Backport from mainline: * gfortranspec.c (lang_specific_driver): Removed deprecation warning for -M. * lang.opt: Add options -M, -MM, -MD, -MMD, -MF, -MG, -MP, -MT, -MQ. * lang-specs.h (CPP_FORWARD_OPTIONS): Add -M* options. * cpp.h (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. * cpp.c (gfc_cpp_option): Add deps* members. (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. (gfc_cpp_init_options): Initialize new options. (gfc_cpp_handle_option): Handle new options. (gfc_cpp_post_options): Map new options to libcpp-options. (gfc_cpp_init): Handle deferred -MQ and -MT options. (gfc_cpp_done): If requested, write dependencies to file. * module.c (gfc_dump_module): Add a module filename as target. * scanner.c (open_included_file): New parameter system; add the included file as dependency. (gfc_open_included_file): Add the included file as dependency. (gfc_open_intrinsic_module): Likewise. * invoke.texi: Removed deprecation warning for -M. * gfortran.texi: Removed Makefile-dependencies project. Modified: branches/gcc-4_4-branch/gcc/fortran/ChangeLog branches/gcc-4_4-branch/gcc/fortran/cpp.c branches/gcc-4_4-branch/gcc/fortran/cpp.h branches/gcc-4_4-branch/gcc/fortran/gfortran.texi branches/gcc-4_4-branch/gcc/fortran/gfortranspec.c branches/gcc-4_4-branch/gcc/fortran/invoke.texi branches/gcc-4_4-branch/gcc/fortran/lang-specs.h branches/gcc-4_4-branch/gcc/fortran/lang.opt branches/gcc-4_4-branch/gcc/fortran/module.c branches/gcc-4_4-branch/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] [4.4 regression] gfortran does not support -Wp, -MD for *.F
--- Comment #21 from jvdelisle at gcc dot gnu dot org 2010-08-08 05:29 --- Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/45175] In gfortran, the calculation of small differences seems to depend on optimization flags
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-08-04 02:42 --- This is a duplicate of PR 323 See http://gcc.gnu.org/bugzilla/show_bug.cgi?id=323 Just use -ffloat-store. *** This bug has been marked as a duplicate of 323 *** -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||DUPLICATE http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45175
[Bug rtl-optimization/323] optimized code gives strange floating point results
--- Comment #137 from jvdelisle at gcc dot gnu dot org 2010-08-04 02:42 --- *** Bug 45175 has been marked as a duplicate of this bug. *** -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added CC||CycleTimeChart at yahoo dot ||com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=323
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #14 from jvdelisle at gcc dot gnu dot org 2010-08-02 13:19 --- Closing as fixed. Regarding comment #7. I do not see the problem on 4.5. I reran tests here on my machine that failed with 4.6 and there were no failures on 4.5 -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-08-01 17:28 --- This appears to be fixed now from the commit of patch in #10. The related failures showing in gcc test results are no longer there. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-07-31 16:28 --- I can see one here on a 32 bit machine. Moving string.h down in the file as noted by David Edelsohn fixes it. I also changed the return type to void in trans-io.c. Index: inquire.c === --- inquire.c (revision 162724) +++ inquire.c (working copy) @@ -26,9 +26,9 @@ /* Implement the non-IOLENGTH variant of the INQUIRY statement */ -#include string.h #include io.h #include unix.h +#include string.h static const char undefined[] = UNDEFINED; -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|NEW |ASSIGNED Last reconfirmed|2010-07-30 09:32:24 |2010-07-31 16:28:49 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-07-31 17:04 --- I see the problem with 4.4.3 as well. I think the patch in #10 is only masking it. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-07-30 14:06 --- #7 confirms my suspicions. I will try to have a look into this in the next few days. If anyone else has time, please do. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45143] [F2008,corrig1] Endless loop with unlimited edit descriptor
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-07-30 14:10 --- Mine -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-07-30 14:10:14 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45143
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-07-30 01:47 --- Reply to comment #9. Yes, this is what I was thinking. I wanted to float the first step out there to see what else we would discover. As seen in PR 45131 there are other oddities we need to deal with. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/45131] [4.6 regression] New Fortran test failures
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-07-30 01:49 --- This is really odd because the patch does not touch these parts of the library, at least for inquire_3.f90. I wonder if there is something latent here. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45131
[Bug fortran/45129] I/O edit descriptors: Warn if the format field is too small for the E and F edit descriptor
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-07-30 01:55 --- I see this as an enhancement. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Severity|normal |enhancement http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45129
[Bug fortran/45108] Namelist read: Not aborted when reading from STDIN
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-07-29 00:59 --- I will have a look. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |jvdelisle at gcc dot gnu dot |dot org |org Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-07-29 00:59:45 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45108
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-07-29 01:32 --- Subject: Bug 44931 Author: jvdelisle Date: Thu Jul 29 01:32:23 2010 New Revision: 162667 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162667 Log: 2010-07-28 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/44931 * io/inquire.c (inquire_via_unit): Use ttyname to return actual device file name for stdin, stdout, and stderr. If ttyname does not succeed fall back to default names for these units. Include string.h to allow using strlen function. * unix.c: Remove typedef of unix_stream structure, move to unix.h. * unix.h: Add typedef of unix_stream structure so that it is accessible to inquire.c. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/inquire.c trunk/libgfortran/io/unix.c trunk/libgfortran/io/unix.h -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/45066] ICE in namelist read in snapshot of 7/24/2010
--- Comment #1 from jvdelisle at gcc dot gnu dot org 2010-07-25 14:26 --- Confirmed. Works with -fno-whole-file. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2010-07-25 14:26:17 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45066
[Bug fortran/45066] ICE in namelist read in snapshot of 7/24/2010
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-07-25 15:04 --- This patchlet at least works for the reduce test case. Assuming there is nothing left to translate. I am not sure this is the right approach, bu the -fdump-tree-original matches for with and without -fno-whole-file. Index: trans-io.c === --- trans-io.c (revision 162507) +++ trans-io.c (working copy) @@ -1499,6 +1499,9 @@ gcc_assert (sym || c); + if (base_addr == NULL) +return; + /* Build the namelist object name. */ string = gfc_build_cstring_const (var_name); -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45066
[Bug fortran/42852] gfortran -Wall warns about truncated lines when only a continuation character is truncated
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-07-25 15:08 --- Subject: Bug 42852 Author: jvdelisle Date: Sun Jul 25 15:07:45 2010 New Revision: 162512 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162512 Log: 2010-07-25 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/42852 * scanner.c (gfc_next_char_literal): Move check for truncation earlier in the function so that it does not get missed by early exits. (load_line): Add checks for quoted strings and free form comments to disable warnings on comments. Add check for ampersand as first character after truncation and don't warn for this case, but warn if there are subsequent non-whitespace characters. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42852
[Bug fortran/42852] gfortran -Wall warns about truncated lines when only a continuation character is truncated
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-07-25 15:35 --- Subject: Bug 42852 Author: jvdelisle Date: Sun Jul 25 15:35:04 2010 New Revision: 162514 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162514 Log: 2010-07-25 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/42852 * gfortran.dg/wtruncate_fix.f: New test. Added: trunk/gcc/testsuite/gfortran.dg/wtruncate_fix.f Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42852
[Bug fortran/42852] gfortran -Wall warns about truncated lines when only a continuation character is truncated
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-07-25 15:38 --- Closing as fixed. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42852
[Bug fortran/45066] ICE in namelist read in snapshot of 7/24/2010
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-07-25 16:21 --- Naturally the patch in #2 breaks everything else for namelists at run time unless I messed up something else here. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45066
[Bug fortran/42852] gfortran -Wall warns about truncated lines when only a continuation character is truncated
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-07-25 17:02 --- I doubt I would be allowed since 4.5.1 is at release candidate 1 phase. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42852
[Bug fortran/45066] ICE in namelist read in snapshot of 7/24/2010
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-07-25 17:37 --- Here is another possibility. Only one namelist regression with this on. Index: trans-io.c === --- trans-io.c (revision 162507) +++ trans-io.c (working copy) @@ -1759,8 +1759,11 @@ build_dt (tree function, gfc_code * code) dt_parm = var; for (nml = dt-namelist-namelist; nml; nml = nml-next) - transfer_namelist_element (block, nml-sym-name, nml-sym, - NULL, NULL); + { + if (nml-sym-backend_decl) + transfer_namelist_element (block, nml-sym-name, nml-sym, + NULL, NULL); + } } else set_parameter_const (block, var, IOPARM_common_flags, mask); -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45066
[Bug fortran/42852] gfortran -Wall warns about truncated lines when only a continuation character is truncated
--- Comment #13 from jvdelisle at gcc dot gnu dot org 2010-07-25 19:10 --- Subject: Bug 42852 Author: jvdelisle Date: Sun Jul 25 19:10:09 2010 New Revision: 162518 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162518 Log: 2010-07-25 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/42852 * scanner.c (gfc_next_char_literal): Enable truncation warning for free-form ''. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42852
[Bug testsuite/44873] array function not fully defined
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-07-25 22:24 --- Generally, the gfortran testsuite test cases are testing specific features and not necessarily strict Standard compliance. There is no need to submit PRs for test cases unless they fail unexpectedly or the specific behavior being tested is accepting wrong results. That does happen occasionally, but I think in this case its a does not matter Also, the compiler is required to detect certain wrong programs, but a lot of the time, it is up to the programmer to get it right or suffer the undefined behavior. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44873
[Bug testsuite/44873] array function not fully defined
--- Comment #4 from jvdelisle at gcc dot gnu dot org 2010-07-25 22:45 --- Closing -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||INVALID http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44873
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #7 from jvdelisle at gcc dot gnu dot org 2010-07-25 00:48 --- test_inquire.f90:17.10: inquire(6,file=str1) 1 Error: INQUIRE statement at (1) cannot contain both FILE and UNIT specifiers However, with a simple modification to the patch the following works. open(6,file=test.dat) inquire(6,name=str1) if (str1 /= test.dat) call abort() -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/40011] Problems with -fwhole-file
--- Comment #69 from jvdelisle at gcc dot gnu dot org 2010-07-25 01:21 --- Great detective work folks! The sneaky bugs usually are simple to fix. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40011
[Bug fortran/41859] ICE on invalid expression involving DT with pointer components in I/O
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-07-23 12:28 --- I was getting back to this and noticed that we no longer ICE on the original test case. I suppose then this has become an accepts invalid or there is something not being initialized and the behavior is random. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41859
[Bug fortran/44931] For INPUT_UNIT, INQUIRE NAME= should not return stdin
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2010-07-24 04:08 --- Created an attachment (id=21296) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21296action=view) Proposed pathc This patch uses ttyname to get the /dev/pts/* name. I have not tried this on a Windows system yet to see what it does. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44931
[Bug fortran/44660] [regression 4.4/4.5/4.6] ICE in resolve_equivalence()
--- Comment #17 from jvdelisle at gcc dot gnu dot org 2010-07-23 03:05 --- With 4.5 about to pop a release it would be good to get this fixed for it, with release manager approval -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44660
[Bug fortran/44697] I/O testsuite failures: \r\n vs \n - gfortran.dg/ftell_3.f90
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2010-07-21 04:21 --- Subject: Bug 44697 Author: jvdelisle Date: Wed Jul 21 04:20:58 2010 New Revision: 162362 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162362 Log: 2010-07-20 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/44697 * gfortran.dg/ftell_3.f90: Take care of cr-lf record endings. Modified: trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/gfortran.dg/ftell_3.f90 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44697
[Bug fortran/44697] I/O testsuite failures: \r\n vs \n - gfortran.dg/ftell_3.f90
--- Comment #6 from jvdelisle at gcc dot gnu dot org 2010-07-21 04:25 --- Closing. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44697
[Bug testsuite/38946] [4.4/4.5/4.6 Regression] gcc trunk 143562 - Testsuite - gfortran failing tests that worked previously
--- Comment #32 from jvdelisle at gcc dot gnu dot org 2010-07-21 04:37 --- Rainer, this is OK to back port. Do you want me to do so? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38946
[Bug fortran/44953] FAIL: gfortran.dg/char4_iunit_1.f03 * execution test
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-07-19 13:12 --- Subject: Bug 44953 Author: jvdelisle Date: Mon Jul 19 13:11:54 2010 New Revision: 162304 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162304 Log: 2010-07-19 Jerry DeLisle jvdeli...@gcc.gnu.org PR libfortran/44953 * io/unix.c (mem_alloc_w4): Return gfc_char4_t instead of char type pointer. (mem_write4): Remove cast to gfc_char4_t. * io/transfer.c (write_block): Use a gfc_char4_t pointer. (memset4): New helper function. (next_record_w): Use new helper function rather than sset for internal units. Don't attempt to pad with spaces if it is not needed. * io/unix.h: Update prototype for mem_alloc_w4. * io/write.c (memset4): Use gfc_char4_t pointer and chracter type. Don't use multiply by 4 to compute offset. (memcpy4): Likewise. (write_default_char4): Use a gfc_char4_t pointer and update memset4 and memcpy calls. (write_a): Likewise. (write_l): Likewise. (write_boz): Likewise. (write_decimal): Likewise. (write_x): Likewise. (write_char): Add support for character(kind=4) internal units that was previously missed. (write_integer): Use a gfc_char4_t pointer and update memset4 and memcpy calls. (write_character): Likewise. (write_separator): Add support for character(kind=4) internal units that was previously missed. * write_float.def (output_float): Use a gfc_char4_t pointer and update memset4 and memcpy calls. (write_infnan): Likewise. (output_float_FMT_G_): Likewise. Modified: trunk/libgfortran/ChangeLog trunk/libgfortran/io/transfer.c trunk/libgfortran/io/unix.c trunk/libgfortran/io/unix.h trunk/libgfortran/io/write.c trunk/libgfortran/io/write_float.def -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44953
[Bug fortran/43851] Add _gfortran_error_stop_numeric
--- Comment #11 from jvdelisle at gcc dot gnu dot org 2010-07-20 01:07 --- Not needed. Closing. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43851
[Bug fortran/37077] Implement Internal Unit I/O for character KIND=4
--- Comment #10 from jvdelisle at gcc dot gnu dot org 2010-07-20 01:09 --- Done. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37077
[Bug fortran/44953] FAIL: gfortran.dg/char4_iunit_1.f03 * execution test
--- Comment #12 from jvdelisle at gcc dot gnu dot org 2010-07-20 01:10 --- Fixed, thanks for report. -- jvdelisle at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44953
[Bug fortran/44697] I/O testsuite failures: \r\n vs \n - gfortran.dg/ftell_3.f90
--- Comment #3 from jvdelisle at gcc dot gnu dot org 2010-07-20 04:35 --- Kai, would you test this for me please. Index: ftell_3.f90 === --- ftell_3.f90 (revision 162291) +++ ftell_3.f90 (working copy) @@ -3,9 +3,10 @@ ! Contributed by Janne Blomqvist, Manfred Schwarb ! and Dominique d'Humieres. program ftell_3 - integer :: i + integer :: i, j + character(1) :: ch character(len=99) :: buffer - open(10, form='formatted', status='scratch', position='rewind') + open(10, form='formatted', position='rewind') write(10, '(a)') '123456' write(10, '(a)') '789' write(10, '(a)') 'CC' @@ -13,16 +14,29 @@ rewind(10) read(10, '(a)') buffer call ftell(10, i) - if(i /= 7) then - call abort() +! Expected: On '\n' systems: 7, on \r\n systems: 8 + if(i /= 7 .and. i /= 8) then +call abort end if read(10,'(a)') buffer if (trim(buffer) /= 789) then call abort() end if - call ftell(10,i) - if (i /= 11) then - call abort() + call ftell(10,j) + close(10) + open(10, access=stream) +! Expected: On '\n' systems: 11, on \r\n systems: 13 + if (i == 7) then +read(10, pos=7) ch +if (ch /= char(10)) call abort +if (j /= 11) call abort end if - close(10) + if (i == 8) then +read(10, pos=7) ch +if (ch /= char(13)) call abort +read(10) ch +if (ch /= char(10)) call abort +if (j /= 13) call abort + end if + close(10, status=delete) end program ftell_3 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44697
[Bug fortran/44953] FAIL: gfortran.dg/char4_iunit_1.f03 * execution test
--- Comment #8 from jvdelisle at gcc dot gnu dot org 2010-07-18 21:07 --- Created an attachment (id=21245) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21245action=view) Proposed Patch Please test the attached patch. This patch cleans up pointer use and adds a few snippets I missed in the first implementations, namely in write_char and write_separator. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44953