motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-07 17:53:56 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

tcg/ppc: Sync tcg_out_test and constraints

Browse source 
Ensure the code structure is the same for matching constraints
and emitting code, lest we allow constants that cannot be
trivially tested.

Cc: qemu-stable@nongnu.org
Fixes: ad788aebba ("tcg/ppc: Support TCG_COND_TST{EQ,NE}")
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2487
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <44328324-af73-4439-9d2b-d414e0e13dd7@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
This commit is contained in:
Richard Henderson 2024-08-07 04:04:18 +00:00
parent 6d00c6f982
commit 682a052805
1 changed files with 10 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

15
tcg/ppc/tcg-target.c.inc
View file

@ -325,9 +325,11 @@ static bool tcg_target_const_match(int64_t sval, int ct,
if ((uval & ~0xffff) == 0 || (uval & ~0xffff0000ull) == 0) { if ((uval & ~0xffff) == 0 || (uval & ~0xffff0000ull) == 0) {
return 1; return 1;
} }
if (TCG_TARGET_REG_BITS == 32 || type == TCG_TYPE_I32 if (uval == (uint32_t)uval && mask_operand(uval, &mb, &me)) {
? mask_operand(uval, &mb, &me) return 1;
: mask64_operand(uval << clz64(uval), &mb, &me)) { }
if (TCG_TARGET_REG_BITS == 64 &&
mask64_operand(uval << clz64(uval), &mb, &me)) {
return 1; return 1;
} }
return 0; return 0;
@ -1749,8 +1751,6 @@ static void tcg_out_test(TCGContext *s, TCGReg dest, TCGReg arg1, TCGArg arg2,
if (type == TCG_TYPE_I32) { if (type == TCG_TYPE_I32) {
arg2 = (uint32_t)arg2; arg2 = (uint32_t)arg2;
} else if (arg2 == (uint32_t)arg2) {
type = TCG_TYPE_I32;
} }
if ((arg2 & ~0xffff) == 0) { if ((arg2 & ~0xffff) == 0) {
@ -1761,12 +1761,11 @@ static void tcg_out_test(TCGContext *s, TCGReg dest, TCGReg arg1, TCGArg arg2,
tcg_out32(s, ANDIS | SAI(arg1, dest, arg2 >> 16)); tcg_out32(s, ANDIS | SAI(arg1, dest, arg2 >> 16));
return; return;
} }
if (TCG_TARGET_REG_BITS == 32 || type == TCG_TYPE_I32) { if (arg2 == (uint32_t)arg2 && mask_operand(arg2, &mb, &me)) {
if (mask_operand(arg2, &mb, &me)) {
tcg_out_rlw_rc(s, RLWINM, dest, arg1, 0, mb, me, rc); tcg_out_rlw_rc(s, RLWINM, dest, arg1, 0, mb, me, rc);
return; return;
} }
} else { if (TCG_TARGET_REG_BITS == 64) {
int sh = clz64(arg2); int sh = clz64(arg2);
if (mask64_operand(arg2 << sh, &mb, &me)) { if (mask64_operand(arg2 << sh, &mb, &me)) {
tcg_out_rld_rc(s, RLDICR, dest, arg1, sh, me, rc); tcg_out_rld_rc(s, RLDICR, dest, arg1, sh, me, rc);