Fix gdb 7.0 compilation on IRIX 5.3

Rainer Orth ro@CeBiTec.Uni-Bielefeld.DE
Mon Nov 23 20:45:00 GMT 2009


Tom Tromey <tromey@redhat.com> writes:

>>>>>> "Rainer" == Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> writes:
>
> Rainer> 2009-11-23  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
> Rainer> 	* dwarf2read.c [HAVE_MMAP] (MAP_FAILED): Define if missing.
>
> Rainer> +#ifndef MAP_FAILED
> Rainer> +#define MAP_FAILED -1
> Rainer> +#endif
>
> I would have expected `(void *) -1' here.  This is ok if it was
> warning-free.  Or the other form is also ok.

I didn't see the warning, and used the cast-free form since gcc had
both.  I notice now that they use casts to char * when MAP_FAILED is
used. 

Could you check this in for me?  I couldn't check out an rw cvs tree from
sourceware.org, although I should have rw access to src.

ro@manam 443 > cvs -d :ext:ro@sourceware.org:/cvs/src co gdb
Permission denied (publickey,gssapi-with-mic).
cvs [checkout aborted]: end of file from server (consult above messages if any)

Btw, the crashes I've observed were already known: PR gdb/10819.

     Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


===================================================================
RCS file: gdb/RCS/dwarf2read.c,v
retrieving revision 1.1
diff -up -r1.1 gdb/dwarf2read.c
--- gdb/dwarf2read.c	2009/09/15 16:09:32	1.1
+++ gdb/dwarf2read.c	2009/11/23 20:36:20
@@ -58,6 +58,9 @@
 #endif
 #ifdef HAVE_MMAP
 #include <sys/mman.h>
+#ifndef MAP_FAILED
+#define MAP_FAILED ((void *) -1)
+#endif
 #endif
 
 #if 0



More information about the Gdb-patches mailing list