[cairo] [PATCH] A set of NEON blitters for Pixman.

Koen Kooi k.kooi at student.utwente.nl
Thu Jun 4 07:57:04 PDT 2009


On 04-06-09 14:18, Jonathan Morton wrote:
>>> Ah, I see the real bug.  It's a one-liner - r5 is being sourced
>>> explicitly instead of the %[width] reference.  It's probably a
>>> copy-paste bug, not spotted because I didn't originally write it.
>>
>> This small piece of code actually happened to have two bugs. In addition to
>> using r5 register incorrectly, it just does not advance the destination
>> pointer (r4 register).
>
> Ah.  I'd forgotten about that subtlety - that the NEON addressing modes
> are more like LDMIA/STMIA addressing modes, and that LDR/STR are
> different.
>
> The two patches attached should, together, fix the problem finally.
I applied 0041-Fix-filler-bug.patch and 0042-Really-fix-filler-bug.patch 
to 3c570a815afb282df01f41acad385ff0e3e33899 and I get:

http://scap.linuxtogo.org/files/bc36cdb818b01ca801927066294fd15f.png

Which looks OK :) Many thanks for your and Siarheis patches!

regards,

Koen



More information about the cairo mailing list