[Bug fortran/43849] Add _gfortran_finalize function to close down the library

[Bug fortran/43849] Add _gfortran_finalize function to close down the library

[jb at gcc dot gnu.org]

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

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jb at gcc dot gnu.org

--- Comment #3 from Janne Blomqvist <jb at gcc dot gnu.org> 2010-11-03 16:31:08 UTC ---
In runtime/main.c we have

static void __attribute__((destructor))
cleanup (void)
{
  close_units ();

  if (please_free_exe_path_when_done)
    free ((char *) exe_path);
}

As this function is marked with the destructor attribute, it will be called
when the library is unloaded during program shutdown. 


It might make sense to add 

fbuf_flush(u);
sflush(u);

to some appropriate place in unit.c(close_unit_1). That is, make sure to flush
the unit as part of the closing procedure.

[Bug fortran/43849] Add _gfortran_finalize function to close down the library

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-12-21
     Ever confirmed|0                           |1

--- Comment #4 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
Is this PR still pertinent (2013-12-21)?

[Bug libfortran/43849] Add _gfortran_finalize function to close down the library

[fxcoudert at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43849

--- Comment #6 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
(In reply to Tobias Burnus from comment #0)
> Currently, _gfortran_abort calls:
>   close_units ();
> Additionally, one should call:
>   flush_all_units ();

close_units() actually flushes all units when closing then.


(In reply to Janne Blomqvist from comment #3)
> cleanup() in runtime/main.c is marked with the destructor attribute, it will be
> called when the library is unloaded during program shutdown. 
> 
> It might make sense to add 
> 
> fbuf_flush(u);
> sflush(u);

Same reply: when we close units, we do flush them. This is already ensured in
the close functions of io/unix.c.


So, to summarize: we already have a finalization function, cleanup(). Should we
export it? I am not so sure, unless we have a real use case.

[Bug libfortran/43849] Add _gfortran_finalize function to close down the library

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43849

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |37336

--- Comment #7 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
> So, to summarize: we already have a finalization function, cleanup().
> Should we export it? I am not so sure, unless we have a real use case.

Could someone answer the question?


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37336
[Bug 37336] [F03] Finish derived-type finalization

[Bug libfortran/43849] Add _gfortran_finalize function to close down the library

[jb at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43849

--- Comment #8 from Janne Blomqvist <jb at gcc dot gnu.org> ---
(In reply to Dominique d'Humieres from comment #7)
> > So, to summarize: we already have a finalization function, cleanup().
> > Should we export it? I am not so sure, unless we have a real use case.
> 
> Could someone answer the question?

IMHO, no there isn't a real use case for exporting the cleanup() function.

[Bug libfortran/43849] Add _gfortran_finalize function to close down the library

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43849

--- Comment #9 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
> IMHO, no there isn't a real use case for exporting the cleanup() function.

Does it mean that this PR could be closed? FIXED or WONTFIX?

[Bug libfortran/43849] Add _gfortran_finalize function to close down the library

[fxcoudert at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43849

Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |WONTFIX

--- Comment #10 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Closing as WONTFIX for now. If a real use case (for example co-open array
related) is raised, we can export cleanup() in the future.

[Bug fortran/43849] New: Add _gfortran_finalize function to close down the library

[burnus at gcc dot gnu dot org]

Currently, gfortran only has start up routines, cf.
http://gcc.gnu.org/onlinedocs/gfortran/Non_002dFortran-Main-Program.html

I think it makes sense to also add shutdown routines.

Currently, _gfortran_abort calls:
  close_units ();
Additionally, one should call:
  flush_all_units ();

For some reasons _gfortran_exit_i(4,8) does not. And, I think it would make
sense to call the unit flushing (via the shutdown routine) also for "STOP" and
when leaving the main program. And non-Fortran main programs should do the same
(cf. link above).

The (externally visible) shutdown routine shall return and leave the exiting of
the program to the caller.


-- 
           Summary: Add _gfortran_finalize function to close down the
                    library
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: burnus at gcc dot gnu dot org


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

[Bug fortran/43849] Add _gfortran_finalize function to close down the library

[burnus at gcc dot gnu dot org]

------- Comment #1 from burnus at gcc dot gnu dot org  2010-04-22 13:19 -------
Thus:

a) Create a function named, e.g., _gfortran_shutdown/_gfortran_finalize, which
calls
   flush_all_units ();
   close_units ();

b) Call it from:
   _gfortran_abort
   _gfortran_stop_numeric;
   _gfortran_stop_string;
   _gfortran_error_stop_string
   _gfortran_error_stop_numeric (not yet existing)
and maybe from the run-time error calls such as
   _gfortran_runtime_error
   _gfortran_runtime_error_at

c) Change trans-decl.c's create_main_function to call it

d) Update gfortran.texi

e) Add a call to the gfortran CAF library for ERROR STOP


-- 


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

[Bug fortran/43849] Add _gfortran_finalize function to close down the library

[burnus at gcc dot gnu dot org]

------- Comment #2 from burnus at gcc dot gnu dot org  2010-04-22 13:29 -------
Forgot to mention that one can then also print the exception status:

"8.4 STOP and ERROR STOP statements" (F2008 FDIS):

"If any exception (cf. 14 Exceptions and IEEE arithmetic) is signaling on that
image, the processor shall issue a warning indicating which exceptions are
signaling; this warning shall be on the unit identified by the named constant
ERROR_UNIT (13.8.2.8). It is recommended that the stop code is made available
by formatted output to the same unit."


-- 


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