[Mono-devel-list] (AMD64) Compiler Warnings (2)

Martin Baulig martin at ximian.com
Tue Dec 7 14:10:37 EST 2004


On Mon, 2004-12-06 at 17:50 +0100, Willibald Krenn wrote:
> >> struct MonoDebugLineNumberEntry {
> >> 	guint32 offset;
> >>-	guint32 address;
> >>+	uintptr_t address;

> Ok, so I'll revert that to guint32 and add a *big* FIXME.
> But this can break the code on 64 bit architectures as long as the debug 
> code uses this field as real address?!

Changing this to something which is not guaranteed to be a 32 bit
unsigned integer _will_ break the debugger.  The structure is read from
the symbol file - and the symbol file contains two 32 bit unsigned
integers on all platforms (so you can copy a symbol file from one
machine to another, just like an assembly).

> >>+#include <intps.h>
> > 
> > mono-debug.h is an installed header and you can't add headers that require config.h 
> > to it.
> 
> That complicates matters. IIRC I need uintptr_t in that header for some 
> function arguments that take some address..

The only change I saw in that file was that additional #include - or do
I miss something ?

Martin





More information about the Mono-devel-list mailing list