[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 11/22] tcg/sparc: Split out target constraints to tcg-targ
From: |
Richard Henderson |
Subject: |
Re: [PATCH v2 11/22] tcg/sparc: Split out target constraints to tcg-target-con-str.h |
Date: |
Tue, 19 Jan 2021 13:29:18 -1000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 1/19/21 4:58 AM, Peter Maydell wrote:
> On Fri, 15 Jan 2021 at 21:14, Richard Henderson
> <richard.henderson@linaro.org> wrote:
>>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>> tcg/sparc/tcg-target-con-str.h | 22 +++++++++++++++++
>> tcg/sparc/tcg-target.h | 5 +---
>> tcg/sparc/tcg-target.c.inc | 45 +++++-----------------------------
>> 3 files changed, 29 insertions(+), 43 deletions(-)
>> create mode 100644 tcg/sparc/tcg-target-con-str.h
>>
>> diff --git a/tcg/sparc/tcg-target-con-str.h b/tcg/sparc/tcg-target-con-str.h
>> new file mode 100644
>> index 0000000000..6dc5b95f33
>> --- /dev/null
>> +++ b/tcg/sparc/tcg-target-con-str.h
>> @@ -0,0 +1,22 @@
>> +/* SPDX-License-Identifier: MIT */
>> +/*
>> + * Define Sparc target-specific operand constraints.
>> + * Copyright (c) 2021 Linaro
>> + */
>> +
>> +/*
>> + * Define constraint letters for register sets:
>> + * REGS(letter, register_mask)
>> + */
>> +REGS('r', 0xffffffff)
>> +REGS('R', ALL_64)
>> +REGS('s', 0xffffffff & ~RESERVE_QLDST)
>> +REGS('S', ALL_64 & ~RESERVE_QLDST)
>
> As with s390, I think an ALL_GENERAL_REGS constant would help here.
>
>
>> +
>> +/*
>> + * Define constraint letters for constants:
>> + * CONST(letter, TCG_CT_CONST_* bit set)
>> + */
>> +CONST('I', TCG_CT_CONST_S11)
>> +CONST('J', TCG_CT_CONST_S13)
>> +CONST('Z', TCG_CT_CONST_ZERO)
>
>
>> -static const char *target_parse_constraint(TCGArgConstraint *ct,
>> - const char *ct_str, TCGType type)
>> -{
>> - switch (*ct_str++) {
>> - case 'r':
>> - ct->regs = 0xffffffff;
>> - break;
>> - case 'R':
>> - ct->regs = ALL_64;
>> - break;
>> - case 'A': /* qemu_ld/st address constraint */
>> - ct->regs = TARGET_LONG_BITS == 64 ? ALL_64 : 0xffffffff;
>> - reserve_helpers:
>> - tcg_regset_reset_reg(ct->regs, TCG_REG_O0);
>> - tcg_regset_reset_reg(ct->regs, TCG_REG_O1);
>> - tcg_regset_reset_reg(ct->regs, TCG_REG_O2);
>> - break;
>
> The "A" constraint seems to have vanished in the conversion...
Argh. Something that would have been found eventually, but I really want to
turn into a build-time error.
r~
- [PATCH v2 02/22] tcg/i386: Move constraint type check to tcg_target_const_match, (continued)
- [PATCH v2 02/22] tcg/i386: Move constraint type check to tcg_target_const_match, Richard Henderson, 2021/01/15
- [PATCH v2 09/22] tcg/riscv: Split out target constraints to tcg-target-con-str.h, Richard Henderson, 2021/01/15
- [PATCH v2 11/22] tcg/sparc: Split out target constraints to tcg-target-con-str.h, Richard Henderson, 2021/01/15
- [PATCH v2 08/22] tcg/mips: Split out target constraints to tcg-target-con-str.h, Richard Henderson, 2021/01/15
- [PATCH v2 10/22] tcg/s390: Split out target constraints to tcg-target-con-str.h, Richard Henderson, 2021/01/15
- [PATCH v2 12/22] tcg: Remove TCG_TARGET_CON_STR_H, Richard Henderson, 2021/01/15
- [PATCH v2 13/22] tcg/i386: Split out constraint sets to tcg-target-con-set.h, Richard Henderson, 2021/01/15