On Fri, 24 Apr 2015 10:25:05 +0300 Pekka Paalanen <[email protected]> wrote:
> On Thu, 23 Apr 2015 20:28:58 +0100 > "Ben Avison" <[email protected]> wrote: > > > On Thu, 23 Apr 2015 13:10:10 +0100, Pekka Paalanen <[email protected]> > > wrote: > > > > > Affine-bench differs from lowlevel-blt-bench in the following: > > > - does not test different sized operations fitting to specific caches, > > > destination is always 1920x1080 > > > - allows defining the affine transformation parameters > > > - carefully computes operation extents to hit the COVER_CLIP fast paths > > I've reviewed your version, looks fine to me. A very minor point: I'm not > > sure it's worth making a copy of the transform struct at the start of > > bench() because we mostly only use a pointer to the struct thereafter, so > > you might as well have kept using bi->transform. > > Thank you, I'll take that as R-b. Pushed. 58e21d3..e0c0153 master -> master Thanks, pq _______________________________________________ Pixman mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pixman
