[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tcg/i386: fix unsigned vector saturating arithm
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-devel] [PATCH] tcg/i386: fix unsigned vector saturating arithmetic |
Date: |
Fri, 8 Feb 2019 18:09:39 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 08/02/2019 17:39, Richard Henderson wrote:
> On 2/7/19 2:42 PM, Mark Cave-Ayland wrote:
>> Due to a cut/paste error in the original implementation, the unsigned vector
>> saturating arithmetic was erroneously being calculated as signed vector
>> saturating
>> arithmetic.
>>
>> Fixes: 8ffafbcec2 ("tcg/i386: Implement vector saturating arithmetic")
>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>> ---
>> tcg/i386/tcg-target.inc.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Ooof. Thanks.
AFAICT none of the other TCG backends currently make use of it, but it was this
bug
causing the graphical corruption that myself and Howard saw testing the PPC
vector
patchset.
FWIW I've now rebased and repushed the outstanding patches from your patchset
along
with this fix to https://github.com/mcayland/qemu/commits/ppc-altivec-v6 as it
would
be great if this could make it into 4.0.
ATB,
Mark.