[Pixman] [PATCH 4/9] armv7: Simplify constant load

Mizuki Asakura ed6e117f at gmail.com
Wed Apr 13 12:24:07 UTC 2016


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 <bavison at 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 <bavison at 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/pixman


More information about the Pixman mailing list