[Bug libfortran/53051] I/O: Support reading floating-point numbers which use Q for the exponent

2012-04-22 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53051

--- Comment #2 from Tobias Burnus burnus at gcc dot gnu.org 2012-04-22 
17:28:39 UTC ---
Author: burnus
Date: Sun Apr 22 17:28:34 2012
New Revision: 186675

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=186675
Log:
2012-04-22  Tobias Burnus  bur...@net-b.de

PR fortran/53051
* io/list_read.c (parse_real): Support q for the
exponential.
* io/read.c (read_f): Ditto.

2012-04-22  Tobias Burnus  bur...@net-b.de

PR fortran/53051
* gfortran.dg/read_float_4.f90: New.


Added:
trunk/gcc/testsuite/gfortran.dg/read_float_4.f90
Modified:
trunk/gcc/testsuite/ChangeLog
trunk/libgfortran/ChangeLog
trunk/libgfortran/io/list_read.c
trunk/libgfortran/io/read.c


[Bug libfortran/53051] I/O: Support reading floating-point numbers which use Q for the exponent

2012-04-22 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53051

Tobias Burnus burnus at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||burnus at gcc dot gnu.org
 Resolution||FIXED

--- Comment #3 from Tobias Burnus burnus at gcc dot gnu.org 2012-04-22 
17:30:02 UTC ---
FIXED on the 4.8 trunk.


[Bug libfortran/53051] I/O: Support reading floating-point numbers which use Q for the exponent

2012-04-20 Thread jb at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53051

Janne Blomqvist jb at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2012-04-20
 CC||jb at gcc dot gnu.org
 Ever Confirmed|0   |1

--- Comment #1 from Janne Blomqvist jb at gcc dot gnu.org 2012-04-20 08:21:05 
UTC ---
The argument against supporting this, I suppose, is that we never generate a
Q|q exponent on output. 

As a result, the example in c.l.f doesn't try to parse something that was
previously written by formatting a floating point number, but rather a
hand-crafted string. IOW, it seems rather artificial.

That being said, it seems ifort supports it, so maybe we need to as well..
*sigh*.