target/arm: Fix offset scaling for LD_zprr and ST_zprr

The scaling should be solely on the memory operation size; the number
of registers being loaded does not come in to the initial computation.

Backports commit 50ef1cbf31caad21019ae6fa8036ed6f29244ba5 from qemu
This commit is contained in:
Richard Henderson 2018-08-17 13:50:58 -04:00 committed by Lioncash
parent 86d6bb4d43
commit 731bcd194b
No known key found for this signature in database
GPG key ID: 4E3C3CC1031BA9C7

View file

@ -4832,8 +4832,7 @@ static bool trans_LD_zprr(DisasContext *s, arg_rprr_load *a, uint32_t insn)
if (sve_access_check(s)) { if (sve_access_check(s)) {
TCGContext *tcg_ctx = s->uc->tcg_ctx; TCGContext *tcg_ctx = s->uc->tcg_ctx;
TCGv_i64 addr = new_tmp_a64(s); TCGv_i64 addr = new_tmp_a64(s);
tcg_gen_muli_i64(tcg_ctx, addr, cpu_reg(s, a->rm), tcg_gen_shli_i64(tcg_ctx, addr, cpu_reg(s, a->rm), dtype_msz(a->dtype));
(a->nreg + 1) << dtype_msz(a->dtype));
tcg_gen_add_i64(tcg_ctx, addr, addr, cpu_reg_sp(s, a->rn)); tcg_gen_add_i64(tcg_ctx, addr, addr, cpu_reg_sp(s, a->rn));
do_ld_zpa(s, a->rd, a->pg, addr, a->dtype, a->nreg); do_ld_zpa(s, a->rd, a->pg, addr, a->dtype, a->nreg);
} }
@ -5074,7 +5073,7 @@ static bool trans_ST_zprr(DisasContext *s, arg_rprr_store *a, uint32_t insn)
if (sve_access_check(s)) { if (sve_access_check(s)) {
TCGContext *tcg_ctx = s->uc->tcg_ctx; TCGContext *tcg_ctx = s->uc->tcg_ctx;
TCGv_i64 addr = new_tmp_a64(s); TCGv_i64 addr = new_tmp_a64(s);
tcg_gen_muli_i64(tcg_ctx, addr, cpu_reg(s, a->rm), (a->nreg + 1) << a->msz); tcg_gen_shli_i64(tcg_ctx, addr, cpu_reg(s, a->rm), a->msz);
tcg_gen_add_i64(tcg_ctx, addr, addr, cpu_reg_sp(s, a->rn)); tcg_gen_add_i64(tcg_ctx, addr, addr, cpu_reg_sp(s, a->rn));
do_st_zpa(s, a->rd, a->pg, addr, a->msz, a->esz, a->nreg); do_st_zpa(s, a->rd, a->pg, addr, a->msz, a->esz, a->nreg);
} }