On 21 May 2022 08:59, Sergei Trofimovich via Gdb-patches wrote:
> --- a/sim/cris/sim-if.c
> +++ b/sim/cris/sim-if.c

these BFD_VMA_FMT changes look fine

> --- a/sim/m32c/syscalls.c
> +++ b/sim/m32c/syscalls.c
>
> -	  printf ("gettimeofday: %ld sec %ld usec to 0x%x\n", tv.tv_sec,
> -		  tv.tv_usec, tvaddr);
> +	  printf ("gettimeofday: %lld sec %lld usec to 0x%x\n",
> +		  (long long)tv.tv_sec, (long long)tv.tv_usec, tvaddr);
>
> --- a/sim/rx/syscalls.c
> +++ b/sim/rx/syscalls.c
>
> -	  printf ("gettimeofday: %ld sec %ld usec to 0x%x\n", tv.tv_sec,
> -		  tv.tv_usec, tvaddr);
> +	  printf ("gettimeofday: %lld sec %lld usec to 0x%x\n",
> +		  (long long)tv.tv_sec, (long long)tv.tv_usec, tvaddr);

i think using explicit 64-bit types (i.e. int64_t) rather than long long
would be better.  it's what we used in other places so far (like the sim
callback APIs).
-mike