qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V3 03/12] Add MIPS DSP Branch instruction Suppor


From: Jia Liu
Subject: Re: [Qemu-devel] [PATCH V3 03/12] Add MIPS DSP Branch instruction Support
Date: Wed, 28 Mar 2012 09:27:33 +0800

On Tue, Mar 27, 2012 at 11:46 PM, Richard Henderson <address@hidden> wrote:
> On 03/27/12 02:24, Jia Liu wrote:
>> Add MIPS DSP Branch instruction Support.
>>
>> Signed-off-by: Jia Liu <address@hidden>
>
> Reviewed-by: Richard Henderson <address@hidden>
>
>> +        case OPC_BPOSGE32:
>> +            tcg_gen_setcondi_tl(TCG_COND_GT, bcond, t0, 31);
>
> Nit: It might be better to use TCG_COND_GE ... 32, since that's
> both the name of the insn and its official documentation.  Of
> course that makes no difference to the correctness.
>

Yeah, TCG_COND_GE looks better, I'll make it.

>
> r~

Regards,
Jia.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]