[Bug bootstrap/39279] New: [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[Bug bootstrap/39279] New: [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[rob1weld at aol dot com]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 2390 bytes --]

# cat stage_current 
stage2

# gcc/xgcc -v
Using built-in specs.
COLLECT_GCC=gcc/xgcc
Target: i386-pc-solaris2.11
Configured with: ../lto_trunk/configure --prefix=/usr/local/lto
--enable-languages=lto,c++ --enable-shared --disable-static --enable-multilib
--with-gnu-as --with-as=/usr/local/bin/as --with-gnu-ld
--with-ld=/usr/local/bin/ld --with-gmp=/usr/local --with-mpfr=/usr/local
--without-ppl --with-demangler-in-ld --enable-clocale=gnu
--with-system-libunwind --with-libelf=/usr/local CPPFLAGS=-I/usr/local/include
Thread model: posix
gcc version 4.4.0 20090218 (experimental) [lto revision 144375] (lto merged
with rev 144262) 


/usr/share/src/lto_build/./prev-gcc/xgcc -B/usr/share/src/lto_build/./prev-gcc/
-B/usr/local/lto/i386-pc-solaris2.11/bin/ -c  -g -O2 -DIN_GCC   -W -Wall
-Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -Wcast-qual
-Wold-style-definition -Wc++-compat -Wmissing-format-attribute -pedantic
-Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -Werror -fno-common
 -DHAVE_CONFIG_H -I. -Ilto -I../../lto_trunk/gcc -I../../lto_trunk/gcc/lto
-I../../lto_trunk/gcc/../include -I../../lto_trunk/gcc/../libcpp/include
-I/usr/local/include -I/usr/local/include -I../../lto_trunk/gcc/../libdecnumber
-I../../lto_trunk/gcc/../libdecnumber/dpd -I../libdecnumber   
../../lto_trunk/gcc/lto/lto.c -o lto/lto.o
cc1: warnings being treated as errors
../../lto_trunk/gcc/lto/lto.c: In function ‘free_section_data’:
../../lto_trunk/gcc/lto/lto.c:497: error: request for implicit conversion from
‘void *’ to ‘caddr_t’ not permitted in C++
gmake[3]: *** [lto/lto.o] Error 1
gmake[3]: Leaving directory `/usr/share/src/lto_build/gcc'
gmake[2]: *** [all-stage2-gcc] Error 2
gmake[2]: Leaving directory `/usr/share/src/lto_build'
gmake[1]: *** [stage2-bubble] Error 2
gmake[1]: Leaving directory `/usr/share/src/lto_build'
gmake: *** [all] Error 2


Rob


-- 
           Summary: [lto] - Werror in ../lto_trunk/gcc/lto/lto.c
           Product: gcc
           Version: lto
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rob1weld at aol dot com
 GCC build triplet: i386-pc-solaris2.11
  GCC host triplet: i386-pc-solaris2.11
GCC target triplet: i386-pc-solaris2.11


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

[Bug lto/39279] [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[rob1weld at aol dot com]

------- Comment #1 from rob1weld at aol dot com  2009-03-09 06:40 -------
Fix:

/* munmap ((void *)computed_offset, computed_len); */
  munmap ((caddr_t)computed_offset, computed_len);

Rob


-- 


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

[Bug lto/39279] [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[bje at gcc dot gnu dot org]

------- Comment #2 from bje at gcc dot gnu dot org  2009-07-09 00:14 -------
Confirmed.  The proposed fix is not correct, though, as the type of the first
argument to munmap _is_ void* according to POSIX.


-- 

bje at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |bje at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-07-09 00:14:25
               date|                            |


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

[Bug lto/39279] [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[bje at gcc dot gnu dot org]

------- Comment #3 from bje at gcc dot gnu dot org  2009-07-15 00:49 -------
Fixed by this patch:

2009-07-09  Rainer Orth  <ro@TechFak.Uni-Bielefeld.DE>

        * lto.c (free_section_data): Cast computed_offset to caddr_t.

I felt this was not the right fix, but I won't argue.


-- 

bje at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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

[Bug lto/39279] [lto] - Werror in ../lto_trunk/gcc/lto/lto.c

[rob1weld at aol dot com]

------- Comment #4 from rob1weld at aol dot com  2009-07-17 06:43 -------
(In reply to comment #2)
> Confirmed.  The proposed fix is not correct, though, as the type of the first
> argument to munmap _is_ void* according to POSIX.

Thanks for applying the patch. I've not looked at 'LTO' source in a few 
months, we could 'cast in the other direction' (fix everything else), 
but I imagine _we_ had a reason not to fix this properly ...

People need to test their code on a few versions of GCC (3.4.x & 4.x.x).

Thanks Ben & Rainer,
Rob


-- 


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