[cairo] [PATCH] Image scaling regression test script and 'fbCompositeSrcScaleNearest' bugfixes

Siarhei Siamashka siarhei.siamashka at nokia.com
Wed May 6 02:30:30 PDT 2009


On Monday 27 April 2009 15:46:13 Siarhei Siamashka wrote:
> On Wednesday 15 April 2009 18:18:18 ext Jeff Muizelaar wrote:
> > On Thu, Apr 02, 2009 at 08:52:59AM +0200, Soeren Sandmann wrote:
> > > Jeff Muizelaar <jeff at infidigm.net> writes:
> > > > > > Jeff also has some ARM optimizations here:
> > > > > >
> > > > > >     http://cgit.freedesktop.org/~jrmuizel/pixman/log/?h=neon
> > > > >
> > > > > Yes, I have seen and tried this patch. Are there any immediate
> > > > > plans for merging it into the main branch?
> > > >
> > > > I hope to merge it soon.
> > >
> > > The common code in it all looks fine to me; I didn't look at the ARM
> > > assembly. Before you merge it, could you split it into smaller commits
> > > that any bugs can be more easily bisected?
> >
> > I've updated:
> > http://cgit.freedesktop.org/~jrmuizel/pixman/log/?h=neon
> >
> > Does that look good to you for merging? Note: there's some new detection
> > code added to pixman-pict for NEON on Linux and Windows CE.
>
> Thanks for committing NEON stuff, that's a nice step forward.
>
> Though looks like it is still missing some updates to configure script so
> that NEON functions can be actually compiled into pixman and used.

Ping.

Having NEON code in repository really does not make much sense at all when it
is never getting compiled into pixman.

Here is a missing part of Ian's patch.

-- 
Best regards,
Siarhei Siamashka
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-ARM-NEON-support-enabled-in-build-system.patch
Type: text/x-diff
Size: 2638 bytes
Desc: not available
Url : http://lists.cairographics.org/archives/cairo/attachments/20090506/c0015b83/attachment.patch 


More information about the cairo mailing list