[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 1/8] s390x/tcg: Implement ADD HALFWORD (AGH)
From: |
David Hildenbrand |
Subject: |
Re: [PATCH v1 1/8] s390x/tcg: Implement ADD HALFWORD (AGH) |
Date: |
Wed, 23 Sep 2020 09:50:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 23.09.20 06:53, Thomas Huth wrote:
> On 22/09/2020 12.31, David Hildenbrand wrote:
>> Easy, just like ADD HALFWORD IMMEDIATE (AGHI).
>>
>> Signed-off-by: David Hildenbrand <david@redhat.com>
>> ---
>> target/s390x/insn-data.def | 1 +
>> target/s390x/translate.c | 1 +
>> 2 files changed, 2 insertions(+)
>>
>> diff --git a/target/s390x/insn-data.def b/target/s390x/insn-data.def
>> index d79ae9e3f1..8dbeaf8c49 100644
>> --- a/target/s390x/insn-data.def
>> +++ b/target/s390x/insn-data.def
>> @@ -52,6 +52,7 @@
>> /* ADD HALFWORD */
>> C(0x4a00, AH, RX_a, Z, r1, m2_16s, new, r1_32, add, adds32)
>> C(0xe37a, AHY, RXY_a, LD, r1, m2_16s, new, r1_32, add, adds32)
>> + C(0xe338, AGH, RXY_a, MIE2,r1, m2_16s, r1, 0, add, adds64)
>> /* ADD HALFWORD IMMEDIATE */
>> C(0xa70a, AHI, RI_a, Z, r1, i2, new, r1_32, add, adds32)
>> C(0xa70b, AGHI, RI_a, Z, r1, i2, r1, 0, add, adds64)
>> diff --git a/target/s390x/translate.c b/target/s390x/translate.c
>> index a777343821..21d77b7e74 100644
>> --- a/target/s390x/translate.c
>> +++ b/target/s390x/translate.c
>> @@ -6098,6 +6098,7 @@ enum DisasInsnEnum {
>> #define FAC_AIS S390_FEAT_ADAPTER_INT_SUPPRESSION
>> #define FAC_V S390_FEAT_VECTOR /* vector facility */
>> #define FAC_VE S390_FEAT_VECTOR_ENH /* vector enhancements
>> facility 1 */
>> +#define FAC_MIE2 S390_FEAT_MISC_INSTRUCTION_EXT /*
>> miscellaneous-instruction-extensions facility 2 */
>
> Maybe you should use "S390_FEAT_MISC_INSTRUCTION_EXT2" (i.e. with 2 at
> the end) to avoid that it gets confused with the first MIE ?
Yes, renaming S390_FEAT_MISC_INSTRUCTION_EXT ->
S390_FEAT_MISC_INSTRUCTION_EXT2 makes sense.
>
> Thomas
>
--
Thanks,
David / dhildenb
- [PATCH v1 0/8] s390x/tcg: Implement some z14 facilities, David Hildenbrand, 2020/09/22
- [PATCH v1 1/8] s390x/tcg: Implement ADD HALFWORD (AGH), David Hildenbrand, 2020/09/22
- [PATCH v1 2/8] s390x/tcg: Implement SUBTRACT HALFWORD (SGH), David Hildenbrand, 2020/09/22
- [PATCH v1 3/8] s390x/tcg: Implement MULTIPLY (MG, MGRK), David Hildenbrand, 2020/09/22
- [PATCH v1 5/8] s390x/tcg: Implement BRANCH INDIRECT ON CONDITION (BIC), David Hildenbrand, 2020/09/22
- [PATCH v1 4/8] s390x/tcg: Implement MULTIPLY HALFWORD (MGH), David Hildenbrand, 2020/09/22