diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2022-11-12 01:15:32 +0700 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2022-12-22 19:29:24 -0500 |
| commit | 63fd5b5ddad9c715531168d5863853017a1f02d3 (patch) | |
| tree | 0ede0e8d44166303d4b451dfc30e096e82e5d505 /sim/sh | |
| parent | 20fea6638f1785b241c39454dcb707234a675524 (diff) | |
| download | binutils-gdb-63fd5b5ddad9c715531168d5863853017a1f02d3.tar.gz | |
sim: switch sim_{read,write} APIs to 64-bit all the time [PR sim/7504]
We've been using SIM_ADDR which has always been 32-bit. This means
the upper 32-bit address range in 64-bit sims is inaccessible. Use
64-bit addresses all the time since we want the APIs to be stable
regardless of the active arch backend (which can be 32 or 64-bit).
The length is also 64-bit because it's completely feasible to have
a program that is larger than 4 GiB in size/image/runtime. Forcing
the caller to manually chunk those accesses up into 4 GiB at a time
doesn't seem useful to anyone.
Bug: https://sourceware.org/PR7504
Diffstat (limited to 'sim/sh')
| -rw-r--r-- | sim/sh/interp.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sim/sh/interp.c b/sim/sh/interp.c index e0cbc7fc6d9..7784ca7aa40 100644 --- a/sim/sh/interp.c +++ b/sim/sh/interp.c @@ -1873,8 +1873,8 @@ sim_resume (SIM_DESC sd, int step, int siggnal) signal (SIGFPE, prev_fpe); } -int -sim_write (SIM_DESC sd, SIM_ADDR addr, const void *buffer, int size) +uint64_t +sim_write (SIM_DESC sd, uint64_t addr, const void *buffer, uint64_t size) { int i; const unsigned char *data = buffer; @@ -1888,8 +1888,8 @@ sim_write (SIM_DESC sd, SIM_ADDR addr, const void *buffer, int size) return size; } -int -sim_read (SIM_DESC sd, SIM_ADDR addr, void *buffer, int size) +uint64_t +sim_read (SIM_DESC sd, uint64_t addr, void *buffer, uint64_t size) { int i; unsigned char *data = buffer; |