This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [gfortran] Fix PR 16861


Andreas Jaeger <aj@suse.de> writes:

> Paul Thomas <paulthomas2@wanadoo.fr> writes:
>
>> [...]
>> Regtested on Cygwin/i686 and FC3/Athlon 1700.
>
> With a complete bootstrap?  CVS head fails to bootstrap now with:
>
> stage1/xgcc -Bstage1/ -B/opt/gcc/4.1-devel/x86_64-suse-linux-gnu/bin/ -c   -g -O2 -DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -pedantic -Wno-long-long -Wno-variadic-macros -Wold-style-definition -Wmissing-format-attribute -Werror -fno-common   -DHAVE_CONFIG_H -I. -Ifortran -I/cvs/gcc/gcc -I/cvs/gcc/gcc/fortran -I/cvs/gcc/gcc/../include -I/cvs/gcc/gcc/../libcpp/include     /cvs/gcc/gcc/fortran/module.c -o fortran/module.o
> cc1: warnings being treated as errors
> /cvs/gcc/gcc/fortran/module.c: In function ‘gfc_use_module’:
> /cvs/gcc/gcc/fortran/module.c:3102: warning: ‘series’ may be used uninitialized in this function
> make[2]: *** [fortran/module.o] Error 1
>
>
>> OK for mainline and 4.03, when open?
>>
>> Paul T
>>
>> ===================================================================
>>
>> 2005-09-15  Paul Thomas  <pault@gcc.gnu.org>
>>
>>    PR fortran/16861
>>    * module.c (read_module): Give symbols from module procedures
>>    different true_name entries to those from the module proper.
>>
>> 2005-09-15  Paul Thomas  <pault@gcc.gnu.org>
>>
>>    PR fortran/16861
>>    * gfortran.dg/nested_modules_2.f90: New test.
>>
>>
>> [...]
>> Index: gcc/gcc/fortran/module.c
>> ===================================================================
>> RCS file: /cvsroot/gcc/gcc/gcc/fortran/module.c,v
>> retrieving revision 1.36
>> diff -c -p -r1.36 module.c
>> *** gcc/gcc/fortran/module.c    9 Sep 2005 00:23:06 -0000    1.36
>> --- gcc/gcc/fortran/module.c    15 Sep 2005 12:42:44 -0000
>> *************** read_module (void)
>> *** 3101,3107 ****
>>    const char *p;
>>    char name[GFC_MAX_SYMBOL_LEN + 1];
>>    gfc_intrinsic_op i;
>> !   int ambiguous, symbol, j, nuse;
>>    pointer_info *info;
>>    gfc_use_rename *u;
>>    gfc_symtree *st;
>> --- 3101,3107 ----
>>    const char *p;
>>    char name[GFC_MAX_SYMBOL_LEN + 1];
>>    gfc_intrinsic_op i;
>> !   int ambiguous, symbol, j, nuse, series;
>>    pointer_info *info;
>>    gfc_use_rename *u;
>>    gfc_symtree *st;
>> *************** read_module (void)
>> *** 3122,3127 ****
>> --- 3122,3128 ----
>>
>>    /* Create the fixup nodes for all the symbols.  */
>>
>> +   series = 0;

This line of the patch was not committed, I'll do this now,

Attachment: pgp00000.pgp
Description: PGP signature

Andreas
-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
  SUSE Linux Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

Attachment: pgp00001.pgp
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]