On Mon, 16 Mar 2015 13:56:53 +0200 Pekka Paalanen <ppaala...@gmail.com> wrote:
> On Tue, 3 Mar 2015 15:24:16 +0000 > Ben Avison <bavi...@riscosopen.org> wrote: > > > Missing "lsl" meant that cases with a 32-bit source and/or mask, and an > > 8-bit destination, the code would not assemble. > > --- > > pixman/pixman-arm-simd-asm.h | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/pixman/pixman-arm-simd-asm.h b/pixman/pixman-arm-simd-asm.h > > index 8de060a..da153c3 100644 > > --- a/pixman/pixman-arm-simd-asm.h > > +++ b/pixman/pixman-arm-simd-asm.h > > @@ -211,8 +211,8 @@ > > PF add, SCRATCH, base, WK0, lsl #bpp_shift-dst_bpp_shift > > PF and, SCRATCH, SCRATCH, #31 > > PF rsb, SCRATCH, SCRATCH, WK0, lsl #bpp_shift-dst_bpp_shift > > - PF sub, SCRATCH, SCRATCH, #1 /* so now ranges are -16..-1 / > > 0..31 / 32..63 */ > > - PF movs, SCRATCH, SCRATCH, #32-6 /* so this sets NC / > > nc / Nc */ > > + PF sub, SCRATCH, SCRATCH, #1 /* so now ranges are > > -16..-1 / 0..31 / 32..63 */ > > + PF movs, SCRATCH, SCRATCH, lsl #32-6 /* so this sets NC > > / nc / Nc */ > > PF bcs, 61f > > PF bpl, 60f > > PF pld, [ptr, #32*(prefetch_distance+2)] > > Hi, > > this one patch looks like it is independent from the series. > > On Sun, 05 Oct 2014 21:03:42 +0200 > soren.sandm...@gmail.com (Søren Sandmann) wrote: > > > ================================================================== > > 0001: Typo in preload Looks good > > ================================================================== > > > > Looks good > > Seems like Søren already agreed. Could this one be pushed alone? Ping? I know there are lots of patches from Ben in the queue, but this small series is the one to be landed first. Thanks, pq _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/pixman