This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: Procedure Pointers: a first patch
- From: Tobias Burnus <tobias dot burnus at physik dot fu-berlin dot de>
- To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>, fortran at gcc dot gnu dot org, Janus Weil <jaydub66 at googlemail dot com>
- Date: Thu, 8 May 2008 06:58:02 +0200
- Subject: Re: Procedure Pointers: a first patch
Paul Richard Thomas wrote:
> To Tobias and FX: I added some debugging functions gfc_show_symbol_n
> and gfc_show_expr_n but these have been removed. There is no
> ChangeLog entry to mark their passing. Do you know when and where they
> went?
This shows the importance of following the GNU ChangeLog style. Some svn
diffing revealed:
r133958 | fxcoudert | 2008-04-06 14:17:33 +0200 (So, 06 Apr 2008) | 13 lines
* dump-parse-tree.c: Use fprintf, fputs and fputc instead of
gfc_status and gfc_status_char. Remove gfc_ prefix of the gfc_show_*
functions and make them static. Add new gfc_dump_parse_tree
function.
But actually this patch does not only remove the gfc_ prefix but removed
the mentioned functions completely. See also:
http://gcc.gnu.org/ml/fortran/2008-04/msg00044.html
> Steven Bosscher wrote:
> > Note how the rhs of the assignment arg0 (INIT_EXPR is gimplified to a
> > simple assignment) is a var decl.
> >
> > For your test case, otoh, I get an ADDR_EXPR on the rhs
Hmm. That's interesting.
Tobias