Hey guys, > Index: sim/erc32/sis.h > =================================================================== > RCS file: /cvs/src/src/sim/erc32/sis.h,v > retrieving revision 1.2 > diff -u -r1.2 sis.h > --- sim/erc32/sis.h 9 Jun 2002 15:45:46 -0000 1.2 > +++ sim/erc32/sis.h 4 May 2010 21:14:55 -0000 > @@ -23,6 +23,7 @@ > #include "ansidecl.h" > #include "gdb/callback.h" > #include "gdb/remote-sim.h" > +#include Unfortunately, this change breaks the build when stdint.h is not available (Eg. sparc-solaris). Apparently, this header was included for 2 things: - define two 64bit types int64 and uint64; - have UINT64_MAX. I assume that the requirement was for 64bit minimum, as opposed to exactly 64bit? Making that assumption, we can remove the need for including stdint.h by using long long instead of int64_t (same thing for the unsigned counterpart). Similarly, UINT64_MAX has a well defined value reguardless of the platform, so it can easily be defined as well. Looking at the rest of the type definitions above, it's actually in line with what's been done so far. Joel: Would that work for you as well? Doug: Would that be OK to commit? I think that the cleanest thing to do here would be to have some configury that would provide our own stdint.h when missing. We already do that for GDB by using gnulib so perhaps one way to do so would be to share the gnulib between GDB and the sim (probably meaning moving it to the root directory). PS: We have the same problem with rx sim, I believe. I haven't tried building it, though. -- Joel