mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-31 14:02:05 -06:00
tcg/i386: Generalize multi-part load overlap test
Test for both base and index; use datahi as a temporary, overwritten by the final load. Always perform the loads in ascending order, so that any (user-only) fault sees the correct address. Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
parent
bf12e2240d
commit
3174941fe0
1 changed files with 15 additions and 16 deletions
|
@ -2221,23 +2221,22 @@ static void tcg_out_qemu_ld_direct(TCGContext *s, TCGReg datalo, TCGReg datahi,
|
||||||
if (TCG_TARGET_REG_BITS == 64) {
|
if (TCG_TARGET_REG_BITS == 64) {
|
||||||
tcg_out_modrm_sib_offset(s, movop + P_REXW + seg, datalo,
|
tcg_out_modrm_sib_offset(s, movop + P_REXW + seg, datalo,
|
||||||
base, index, 0, ofs);
|
base, index, 0, ofs);
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
if (use_movbe) {
|
||||||
|
TCGReg t = datalo;
|
||||||
|
datalo = datahi;
|
||||||
|
datahi = t;
|
||||||
|
}
|
||||||
|
if (base == datalo || index == datalo) {
|
||||||
|
tcg_out_modrm_sib_offset(s, OPC_LEA, datahi, base, index, 0, ofs);
|
||||||
|
tcg_out_modrm_offset(s, movop + seg, datalo, datahi, 0);
|
||||||
|
tcg_out_modrm_offset(s, movop + seg, datahi, datahi, 4);
|
||||||
} else {
|
} else {
|
||||||
if (use_movbe) {
|
tcg_out_modrm_sib_offset(s, movop + seg, datalo,
|
||||||
TCGReg t = datalo;
|
base, index, 0, ofs);
|
||||||
datalo = datahi;
|
tcg_out_modrm_sib_offset(s, movop + seg, datahi,
|
||||||
datahi = t;
|
base, index, 0, ofs + 4);
|
||||||
}
|
|
||||||
if (base != datalo) {
|
|
||||||
tcg_out_modrm_sib_offset(s, movop + seg, datalo,
|
|
||||||
base, index, 0, ofs);
|
|
||||||
tcg_out_modrm_sib_offset(s, movop + seg, datahi,
|
|
||||||
base, index, 0, ofs + 4);
|
|
||||||
} else {
|
|
||||||
tcg_out_modrm_sib_offset(s, movop + seg, datahi,
|
|
||||||
base, index, 0, ofs + 4);
|
|
||||||
tcg_out_modrm_sib_offset(s, movop + seg, datalo,
|
|
||||||
base, index, 0, ofs);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue