Hi. In aarch64, I cannot find corresponding instruction of this patch, but
> movi v2.2s, 0xff, lsl #24 works well as your modification. On 11 April 2016 at 21:26, Ben Avison <bavi...@riscosopen.org> wrote: > A minor point, but 0xFF000000 is already a valid immediate constant for > NEON, there's no need to construct it in two steps. > > Signed-off-by: Ben Avison <bavi...@riscosopen.org> > --- > pixman/pixman-arm-neon-asm.S | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/pixman/pixman-arm-neon-asm.S b/pixman/pixman-arm-neon-asm.S > index 9a5d85a..97315d4 100644 > --- a/pixman/pixman-arm-neon-asm.S > +++ b/pixman/pixman-arm-neon-asm.S > @@ -1186,8 +1186,7 @@ generate_composite_function \ > .endm > > .macro pixman_composite_src_x888_8888_init > - vmov.u8 q2, #0xFF > - vshl.u32 q2, q2, #24 > + vmov.u32 q2, #0xFF000000 > .endm > > generate_composite_function \ > -- > 1.7.5.4 > > _______________________________________________ > Pixman mailing list > Pixman@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/pixman _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/pixman