[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 12/12] [RESEND] target-arm: fix neon shift helpe
From: |
Juha.Riihimaki |
Subject: |
Re: [Qemu-devel] [PATCH 12/12] [RESEND] target-arm: fix neon shift helper functions |
Date: |
Thu, 22 Oct 2009 10:46:37 +0200 |
On Oct 22, 2009, at 10:57, ext Laurent Desnogues wrote:
> On Wed, Oct 21, 2009 at 1:01 PM, <address@hidden> wrote:
>> Current code is broken at least on gcc 4.2, the result of a
>> comparison
>> "-1 >= sizeof(type) * 8" results true and causes wrong code path to
>> be
>> taken. The fix utilizes abs() function where applicable and otherwise
>> adds a test to ensure both arguments are positive before making the
>> aforementioned comparison.
>
> Why don't you instead cast sizeof to int?
No reason, I'll make the v2 patch series use cast instead. It seems to
work just as well.
Juha