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

2009-07-17 Thread 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

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

2009-07-14 Thread 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  r...@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

2009-07-08 Thread 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|-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

2009-03-09 Thread 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