Re: [patch] sim/erc32/ max simulation time extended by using 64bit ints

[Joel Sherrill <joel.sherrill@oarcorp.com>]

On 05/16/2010 08:57 PM, Joel Brobecker wrote:
> 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<stdint.h>
>>      
> 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?
>    
Sure. I don't have a problem with that.

RTEMS has all the C99 types and we are just in the habit of using them.

--joel
> 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.
>
>