diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2024-07-24 14:53:38 +1000 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2024-07-28 14:13:05 +1000 |
| commit | 83340193b991e7a974f117baa86a04db1fd835a9 (patch) | |
| tree | 844301a178ab43d7ec2e94dcfc130028a3eff380 | |
| parent | 93b799fafd9170da3a79a533ea6f73a18de82e22 (diff) | |
| download | qemu-83340193b991e7a974f117baa86a04db1fd835a9.zip qemu-83340193b991e7a974f117baa86a04db1fd835a9.tar.gz qemu-83340193b991e7a974f117baa86a04db1fd835a9.tar.bz2 | |
target/rx: Use target_ulong for address in LI
Using int32_t meant that the address was sign-extended to uint64_t
when passing to translator_ld*, triggering an assert.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2453
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Tested-by: Thomas Huth <thuth@redhat.com>
| -rw-r--r-- | target/rx/translate.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/target/rx/translate.c b/target/rx/translate.c index 9b81cf2..9aade2b 100644 --- a/target/rx/translate.c +++ b/target/rx/translate.c @@ -85,7 +85,8 @@ static uint32_t decode_load_bytes(DisasContext *ctx, uint32_t insn, static uint32_t li(DisasContext *ctx, int sz) { - int32_t tmp, addr; + target_ulong addr; + uint32_t tmp; CPURXState *env = ctx->env; addr = ctx->base.pc_next; |