qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [Qemu-ppc] [PATCH 06/17] ppc: use CRF_* in int_helper.c


From: Alexander Graf
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 06/17] ppc: use CRF_* in int_helper.c
Date: Fri, 05 Sep 2014 09:12:42 +0200
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Thunderbird/31.0


On 03.09.14 20:28, Tom Musta wrote:
> On 8/28/2014 12:15 PM, Paolo Bonzini wrote:
>> Signed-off-by: Paolo Bonzini <address@hidden>
>> ---
>>  target-ppc/int_helper.c | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/target-ppc/int_helper.c b/target-ppc/int_helper.c
>> index f6e8846..9c1c5cd 100644
>> --- a/target-ppc/int_helper.c
>> +++ b/target-ppc/int_helper.c
>> @@ -2303,25 +2303,25 @@ uint32_t helper_bcdadd(ppc_avr_t *r,  ppc_avr_t *a, 
>> ppc_avr_t *b, uint32_t ps)
>>          if (sgna == sgnb) {
>>              result.u8[BCD_DIG_BYTE(0)] = bcd_preferred_sgn(sgna, ps);
>>              zero = bcd_add_mag(&result, a, b, &invalid, &overflow);
>> -            cr = (sgna > 0) ? 4 : 8;
>> +            cr = (sgna > 0) ? 1 << CRF_GT : 1 << CRF_LT;
>>          } else if (bcd_cmp_mag(a, b) > 0) {
>>              result.u8[BCD_DIG_BYTE(0)] = bcd_preferred_sgn(sgna, ps);
>>              zero = bcd_sub_mag(&result, a, b, &invalid, &overflow);
>> -            cr = (sgna > 0) ? 4 : 8;
>> +            cr = (sgna > 0) ? 1 << CRF_GT : 1 << CRF_LT;
>>          } else {
>>              result.u8[BCD_DIG_BYTE(0)] = bcd_preferred_sgn(sgnb, ps);
>>              zero = bcd_sub_mag(&result, b, a, &invalid, &overflow);
>> -            cr = (sgnb > 0) ? 4 : 8;
>> +            cr = (sgnb > 0) ? 1 << CRF_GT : 1 << CRF_LT;
>>          }
>>      }
>>  
>>      if (unlikely(invalid)) {
>>          result.u64[HI_IDX] = result.u64[LO_IDX] = -1;
>> -        cr = 1;
>> +        cr = 1 << CRF_SO;
>>      } else if (overflow) {
>> -        cr |= 1;
>> +        cr |= 1 << CRF_SO;
>>      } else if (zero) {
>> -        cr = 2;
>> +        cr = 1 << CRF_EQ;
>>      }
>>  
>>      *r = result;
>>
> 
> Reviewed-by: Tom Musta <address@hidden>
> Tested-by: Tom Musta <address@hidden>

Thanks, applied to ppc-next.


Alex



reply via email to

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