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

Ben Avison bavison at riscosopen.org
Mon Apr 11 12:26:25 UTC 2016


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



More information about the Pixman mailing list