This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
building warnings using gcc HEAD [Was: Results for 2.15.94 20041024 testsuite on i686-pc-linux-gnu]
- From: Christian Joensson <christian at j-son dot org>
- To: binutils <binutils at sources dot redhat dot com>
- Date: Sun, 24 Oct 2004 17:29:25 +0200
- Subject: building warnings using gcc HEAD [Was: Results for 2.15.94 20041024 testsuite on i686-pc-linux-gnu]
- References: <20041024152624.GA13937@fw.j-son.org>
On Sun, Oct 24, 2004 at 05:26:24PM +0200, Christian Joensson wrote:
> Fedora Core release Rawhide (Rawhide) ia32 pentium III dual i686:
>
> binutils-2.15.92.0.2-5
> bison-1.875c-2
> dejagnu-1.4.4-2
> expect-5.42.1-1
> gcc-3.4.2-6
> gcc4-4.0.0-0.8
> glibc-2.3.3-73
> glibc-headers-2.3.3-73
> glibc-kernheaders-2.4-9.1.87
> kernel-2.6.9-1.639
> kernel-2.6.9-1.640
> kernel-smp-2.6.9-1.639
> kernel-smp-2.6.9-1.640
> tcl-8.4.7-2
>
> LAST_UPDATED: Sun Oct 24 15:01:09 UTC 2004
<snip>
> version: 2.15.94 20041024
> Platform: i686-pc-linux-gnu
> configure flags: --enable-__cxa_atexit --enable-shared
> Counting all warnings,
> there are 148 warnings in stage0 of this bootstrap.
>
> Number of warnings per file:
> 35 /usr/local/src/trunk/src/bfd/elf-eh-frame.c
> 22 /usr/local/src/trunk/src/bfd/peicode.h
> 14 /usr/local/src/trunk/src/binutils/readelf.c
> 10 /usr/local/src/trunk/src/binutils/bucomm.c
> 10 libiberty/concat.c
> 8 /usr/local/src/trunk/src/bfd/merge.c
> 6 /usr/local/src/trunk/src/ld/ldexp.c
> 6 libiberty/regex.c
> 4 /usr/local/src/trunk/src/gprof/gmon_io.c
> 4 /usr/local/src/trunk/src/bfd/opncls.c
> 4 /usr/local/src/trunk/src/bfd/elf.c
> 3 /usr/local/src/trunk/src/bfd/dwarf2.c
> 2 /usr/local/src/trunk/src/gprof/hist.c
> 2 /usr/local/src/trunk/src/bfd/srec.c
> 2 /usr/local/src/trunk/src/bfd/elf-strtab.c
> 2 /usr/local/src/trunk/src/bfd/cofflink.c
> 2 /usr/local/src/trunk/src/bfd/coffgen.c
> 2 libiberty/pex-unix.c
> 2 include/xregex2.h
> 1 /usr/local/src/trunk/src/ld/ldwrite.c
> 1 /usr/local/src/trunk/src/gprof/basic_blocks.c
> 1 /usr/local/src/trunk/src/gas/read.c
> 1 /usr/local/src/trunk/src/gas/config/obj-elf.c
> 1 /usr/local/src/trunk/src/binutils/stabs.c
> 1 /usr/local/src/trunk/src/binutils/objdump.c
> 1 /usr/local/src/trunk/src/bfd/elfcode.h
> 1 lex.yy.c
>
> Number of warning types:
> 25 pointer targets in assignment differ in signedness
> 20 pointer targets in passing argument 1 of 'strlen' differ in signedness
> 10 traditional C rejects ISO C style function definitions
> 10 the use of \`???' is dangerous, better use \`???'
> 8 pointer targets in passing argument 1 of '__builtin_strcmp' differ in signedness
> 7 pointer targets in passing argument 2 of 'read_unsigned_leb128' differ in signedness
> 6 signed and unsigned type in conditional expression
> 6 pointer targets in passing argument 2 of 'gmon_io_read_32' differ in signedness
> 6 pointer targets in initialization differ in signedness
> 5 pointer targets in passing argument 2 of 'read_leb128' differ in signedness
> 4 pointer targets in passing argument 2 of '__builtin_strcmp' differ in signedness
> 3 pointer targets in passing argument 3 of 'pe_ILF_make_a_symbol' differ in signedness
> 3 pointer targets in passing argument 2 of 'sec_merge_add' differ in signedness
> 3 pointer targets in passing argument 1 of '__builtin_strchr' differ in signedness
> 2 variable 'out' might be clobbered by 'longjmp' or 'vfork'
> 2 pointer targets in passing argument 2 of 'read_signed_leb128' differ in signedness
> 2 pointer targets in passing argument 2 of 'bfd_calc_gnu_debuglink_crc32' differ in signedness
> 2 pointer targets in passing argument 1 of 'strcpy' differ in signedness
> 2 pointer targets in passing argument 1 of 'byte_get' differ in signedness
> 2 ISO C90 does not support 'static' or type qualifiers in parameter array declarators
> 1 'yyunput' defined but not used
> 1 'varargs' may be used uninitialized in this function
> 1 'strtab' may be used uninitialized in this function
> 1 'result\$value' is used uninitialized in this function
> 1 'result\$valid_p' may be used uninitialized in this function
> 1 'result\$str' is used uninitialized in this function
> 1 'result\$section' is used uninitialized in this function
> 1 pointer targets in return differ in signedness
> 1 pointer targets in passing argument 5 of 'srec_write_record' differ in signedness
> 1 pointer targets in passing argument 4 of 'srec_write_record' differ in signedness
> 1 pointer targets in passing argument 3 of 'coff_write_native_symbol' differ in signedness
> 1 pointer targets in passing argument 3 of 'coff_write_alien_symbol' differ in signedness
> 1 pointer targets in passing argument 3 of 'bfd_simple_get_relocated_section_contents' differ in signedness
> 1 pointer targets in passing argument 2 of 'strcpy' differ in signedness
> 1 pointer targets in passing argument 2 of 'sec_merge_hash_lookup' differ in signedness
> 1 pointer targets in passing argument 2 of 'check_eh_frame' differ in signedness
> 1 pointer targets in passing argument 1 of 'strrchr' differ in signedness
> 1 'new\$str' is used uninitialized in this function
> 1 'hdr\$length' may be used uninitialized in this function
> 1 'count' may be used uninitialized in this function
There's quite a potential for code improvements here... note that this
was compiled using the gcc HEAD as of Fedora Core Development
(rawhide).
Cheers,
/ChJ