[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-29 Thread jvdelisle at gcc dot gnu dot org


--- Comment #13 from jvdelisle at gcc dot gnu dot org  2008-12-29 20:47 
---
I will try to close this again. :)


-- 

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=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-28 Thread mikael at gcc dot gnu dot org


--- Comment #12 from mikael at gcc dot gnu dot org  2008-12-28 19:19 ---
(In reply to comment #11)
> This test case runs fine here, maybe your trunk is not fully updated?
It works with a fresh checkout and a fresh bootstrap.
My trunk was up to date; it was probably the dirty build tree trap again. 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-27 Thread jvdelisle at gcc dot gnu dot org


--- Comment #11 from jvdelisle at gcc dot gnu dot org  2008-12-28 03:53 
---
Something is amiss. Yes the change to fmt_g0_1.f08 is necessary.  The output
should have no blank spaces in the fields for floating point.

This test case runs fine here, maybe your trunk is not fully updated?

$ cat fmt_g0_1.f08 
character(25) :: string = "(g0,g0,g0)" 
character(33) :: buffer
write(*, '(g0,g0,g0)') ':',12340,':'
write(*, string) ':',0,':'
write(*, string) ':',1.0/3.0,':'
write(*, '(1x,a,g0,a)') ':',1.0/3.0,':'
write(*, string) ':',"hello",':'
write(*, "(g0,g0,g0,g0)") ':',.true.,.false.,':'
write(*, "(g0,g0,',',g0,g0)") '(',( 1.2345, 2.4567 ),')'
end

$ ./a.out 
:12340:
:0:
:.3334:
 :.3334:
:hello:
:TF:
(1.2345001,2.4567001)
$


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-27 Thread mikael at gcc dot gnu dot org


--- Comment #10 from mikael at gcc dot gnu dot org  2008-12-27 23:05 ---
(In reply to comment #9)
> Closing, fixed on 4.4
> 
Not yet ;-)

I'm at revision 142934, and I get this on x86_64-unknown-linux-gnu:
FAIL: gfortran.dg/fmt_g0_1.f08  -O0  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O1  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O2  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O3 -fomit-frame-pointer  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O3 -fomit-frame-pointer -funroll-loops 
execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O3 -fomit-frame-pointer -funroll-all-loops
-finline-functions  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -O3 -g  execution test
FAIL: gfortran.dg/fmt_g0_1.f08  -Os  execution test



here is the ouput of the various writes in fmt_g0_1.f08:
:12340:
:0:
: 0.3334:
 : 0.3334:
:hello:
:TF:
(  1.2345001,  2.4567001)


Was the commit in comment #8 needed ?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-23 Thread jvdelisle at gcc dot gnu dot org


--- Comment #9 from jvdelisle at gcc dot gnu dot org  2008-12-23 23:43 
---
Closing, fixed on 4.4


-- 

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 |2008-12-23 23:43:09
   date||
   Target Milestone|--- |4.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-21 Thread jvdelisle at gcc dot gnu dot org


--- Comment #8 from jvdelisle at gcc dot gnu dot org  2008-12-21 22:08 
---
Subject: Bug 38398

Author: jvdelisle
Date: Sun Dec 21 22:06:34 2008
New Revision: 142876

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142876
Log:
2008-12-21  Jerry DeLisle  

PR libfortran/38398
* gfortran.dg/fmt_g0_1.f08: Revise.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gfortran.dg/fmt_g0_1.f08


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-21 Thread jvdelisle at gcc dot gnu dot org


--- Comment #7 from jvdelisle at gcc dot gnu dot org  2008-12-21 21:25 
---
Subject: Bug 38398

Author: jvdelisle
Date: Sun Dec 21 21:23:52 2008
New Revision: 142871

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142871
Log:
2008-12-21  Jerry DeLisle  

PR libfortran/38398
* io/io.h (st_parameter_dt): Add new bit to keep track of when to
suppress blanks for g0 formatting.
* io/transfer.c (formatted_transfer_scalar): Always call write_real_g0
for g0 formatting.
* io.c (write.c): Do not use ES formatting and use new bit to suppress
blanks.
* io/write_float.def (output_float): Adjust the location of setting the
width so that it can be adjusted when suppressing blanks.  Set number
of
blanks to zero when dtp->u.p.g0_no_blanks is set. Do some minor code
clean-up and add some white space for readability.

Modified:
trunk/libgfortran/ChangeLog
trunk/libgfortran/io/io.h
trunk/libgfortran/io/transfer.c
trunk/libgfortran/io/write.c
trunk/libgfortran/io/write_float.def


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-21 Thread jvdelisle at gcc dot gnu dot org


--- Comment #6 from jvdelisle at gcc dot gnu dot org  2008-12-21 21:21 
---
Subject: Bug 38398

Author: jvdelisle
Date: Sun Dec 21 21:20:29 2008
New Revision: 142870

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142870
Log:
2008-12-21  Jerry DeLisle  

PR fortran/38398
* io.c: Add error checks for g0 formatting and provide adjustment of
error loci for improved error messages.

Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/io.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-20 Thread jvdelisle at gcc dot gnu dot org


--- Comment #5 from jvdelisle at gcc dot gnu dot org  2008-12-20 18:34 
---
Created an attachment (id=16950)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16950&action=view)
Revised patch

This is latest version of the patch.  Please test.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-13 Thread jvdelisle at gcc dot gnu dot org


--- Comment #4 from jvdelisle at gcc dot gnu dot org  2008-12-13 23:54 
---
Let's try this again.  I need verification that I am interpreting the comments
correctly.  With this:

! { dg-do compile }
! { dg-options "-std=f2008" }
! PR36725 Compile time error for g0 edit descriptor
write(*, '(g0.3)') 0.1
write(*, '(g0.9)') 1.0
write(*, '(g0.5)') 29.23
write(*, '(g0.8)') -28.4
write(*, '(5(g0.8,1x))') -0.0001, 2.307, ">",3.45,"<"
write(*, '(2(f0.8,1x))') -0.0001, 2.307
write(*, '(a,g10.4,a)') ">",3.45,"<"
write(*, '(a,g0.4,a)') ">",3.45,"<"
write(*, '(a,f0.4,a)') ">",3.45,"<"
end

I get the following output:

$ ./a.out 
.100
1.0
29.23000
-28.3962
-.0001 2.3062 > 3.4505 <
-.0001 2.3062
> 3.450<
>3.4500<
>3.4500<


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-13 Thread jvdelisle at gcc dot gnu dot org


--- Comment #3 from jvdelisle at gcc dot gnu dot org  2008-12-13 21:19 
---
Disregard comment #2, I found the relevant text describing this that I needed.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-13 Thread jvdelisle at gcc dot gnu dot org


--- Comment #2 from jvdelisle at gcc dot gnu dot org  2008-12-13 21:08 
---
Please confirm.  Is the output for this correct?

write(*, '(g0.3)') 0.1
write(*, '(g0.9)') 1.0
write(*, '(g0.5)') 29.23
write(*, '(g0.8)') -28.4
write(*, '(g0.8)') -0.0001
write(*, '(a,g10.4,a)') ">",3.45,"<"
write(*, '(g0.4)') 3.45
end

  0.100
1.0
   29.23000
   -28.3962
-0.0001
> 3.450<
 3.4500


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398



[Bug fortran/38398] g0.w edit descriptor: Update for F2008 Tokyo meeting changes

2008-12-04 Thread burnus at gcc dot gnu dot org


--- Comment #1 from burnus at gcc dot gnu dot org  2008-12-04 12:32 ---
Jerry, I've created this PR to make sure it won't get forgotten.


-- 

burnus at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||jvdelisle at gcc dot gnu dot
   ||org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38398