This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: PR32634 -- reverted patch, gfortran.dg/interface_16.f90 failure
- From: Tobias Burnus <burnus at net-b dot de>
- To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- Cc: Daniel Franke <franke dot daniel at gmail dot com>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches List <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 12 Jul 2007 21:28:18 +0200
- Subject: Re: PR32634 -- reverted patch, gfortran.dg/interface_16.f90 failure
- References: <339c37f20707120450y7ae2d641ucb173423d7486857@mail.gmail.com> <339c37f20707121207s2dcc404djc4910f840ebb43b9@mail.gmail.com>
Hi Paul,
Paul Richard Thomas wrote:
> nuse = number_use_names (sym->name);
> if (nuse == 0)
> ! {
> ! mio_symbol_interface (&sym->name, &sym->module, &sym->generic);
> ! return;
> ! }
Ignoring the fact that there seems to be some whitespace problem (spaces
vs. tabs), this does not compile here:
gcc/fortran/module.c: In function 'read_module':
gcc/fortran/module.c:3465: error: 'sym' may be used uninitialized in
this function
We have:
gfc_symbol *sym; [...]
while (peek_atom () != ATOM_RPAREN)
{ [...]
sym = find_true_name (info->u.rsym.true_name, info->u.rsym.module);
} [...]
while (peek_atom () != ATOM_RPAREN)
{ [...]
mio_symbol_interface (&sym->name, &sym->module, &sym->generic);
I don't quickly see whether gcc is overly strict or right.
Tobias