[Pixman] fast-scale branch performance improvements
Alexander Larsson
alexl at redhat.com
Tue Mar 16 04:25:33 PDT 2010
On Tue, 2010-03-16 at 12:10 +0200, Siarhei Siamashka wrote:
> On Monday 15 March 2010, Alexander Larsson wrote:
> > On Mon, 2010-03-15 at 03:25 +0200, Siarhei Siamashka wrote:
> > > But before it gets committed, some problems with potential fixed
> point
> > > overflows when dealing with large images need to be addressed. I
> have
> > > made a test program which can expose some of these problems:
> > >
> http://cgit.freedesktop.org/~siamashka/pixman/log/?h=largescaling-test
> >
> > I pushed a new version of my branch, rebased on:
> > http://cgit.freedesktop.org/~sandmann/pixman/log/?h=alex-scaler
> > on (as before):
> > http://cgit.freedesktop.org/~alexl/pixman/log/?h=alex-scaler
> >
> > This has a new flag FAST_PATH_16BIT_SAFE which handles this problem
> (and
> > it runs your test successfully.
>
> Now it seems to fail scaling-test.
Ugh. here too. I wonder how that broke? Let me look at it
> Also gcc shows the following warnings:
>
> pixman-fast-path.c: In
> function ‘fast_composite_scaled_nearest_x888_565_SRC_none’:
> pixman-fast-path.c:1560: warning: integer constant is too large for
> ‘long’
> type
> pixman-fast-path.c:1560: warning: integer constant is too large for
> ‘long’
> type
> pixman-fast-path.c:1560: warning: integer constant is too large for
> ‘long’
> type
> pixman-fast-path.c: In
> function ‘fast_composite_scaled_nearest_x888_565_SRC_normal’:
> pixman-fast-path.c:1561: warning: integer constant is too large for
> ‘long’
> type
> pixman-fast-path.c:1561: warning: integer constant is too large for
> ‘long’
> type
> pixman-fast-path.c:1561: warning: integer constant is too large for
> ‘long’
> type
Macros make this sort of painful to debug...
--
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Alexander Larsson Red Hat, Inc
alexl at redhat.com alexander.larsson at gmail.com
He's a suave native American stage actor with no name. She's a tortured
hip-hop single mother from the wrong side of the tracks. They fight crime!
More information about the Pixman
mailing list