[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] add MIPS DSP helpers define
From: |
Jia Liu |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] add MIPS DSP helpers define |
Date: |
Thu, 22 Mar 2012 21:45:00 +0800 |
On Wed, Mar 21, 2012 at 4:17 PM, Stefan Weil <address@hidden> wrote:
> Am 12.03.2012 09:32, schrieb Jia Liu:
>
>>
>> This patch is the helper define of MIPS ASE DSP.
>>
>> Signed-off-by: Jia Liu <address@hidden>
>> ---
>> target-mips/helper.h | 152
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>> 1 files changed, 152 insertions(+), 0 deletions(-)
>>
>> diff --git a/target-mips/helper.h b/target-mips/helper.h
>> index 442f684..1abf582 100644
>> --- a/target-mips/helper.h
>> +++ b/target-mips/helper.h
>> @@ -297,4 +297,156 @@ DEF_HELPER_0(rdhwr_ccres, tl)
>> DEF_HELPER_1(pmon, void, int)
>> DEF_HELPER_0(wait, void)
>>
>> +/* MIPS32 DSP */
>> +DEF_HELPER_1(absqsph, i32, i32)
>> +DEF_HELPER_1(absqsw, i32, i32)
>> +DEF_HELPER_2(addqph, i32, i32, i32)
>> +DEF_HELPER_2(addqsph, i32, i32, i32)
>
> [snip]
>
> Hi,
>
> I know that a lot of people love such tabulated code, but I personally
> would prefer to see those DEF_HELPER lines using the style which
> is used in the existing code (one space after comma).
>
Hi Stefan,
Thank you for review.
I've fixed it, addressed your comments.
Regards,
Jia.
> Regards,
> Stefan Weil
>
- [Qemu-devel] [PATCH 0/4] MIPS ASE DSP Support for Qemu, Jia Liu, 2012/03/12
- [Qemu-devel] [PATCH 1/4] add MIPS DSP helpers define, Jia Liu, 2012/03/12
- [Qemu-devel] [PATCH 2/4] add MIPS DSP helpers implement, Jia Liu, 2012/03/12
- [Qemu-devel] [PATCH 3/4] add MIPS DSP translation, Jia Liu, 2012/03/12
- [Qemu-devel] [PATCH 4/4] add MIPS DSP testcase, Jia Liu, 2012/03/12
- Re: [Qemu-devel] [PATCH 0/4] MIPS ASE DSP Support for Qemu, Jia Liu, 2012/03/20