[x264-devel] [PATCH] arm: Implement luma intra deblocking
Martin Storsjö
martin at martin.st
Tue Sep 1 15:47:26 CEST 2015
checkasm timing Cortex-A7 A8 A9
deblock_luma_intra[0]_c 5988 4653 4316
deblock_luma_intra[0]_neon 3217 2153 2227
deblock_luma_intra[1]_c 7119 5905 5347
deblock_luma_intra[1]_neon 2153 1394 1486
This includes extra optimizations by Janne Grunau.
Timings from a separate build, on Exynos 5422:
Cortex-A7 A15
deblock_luma_intra[0]_c 6627 3300
deblock_luma_intra[0]_neon 3236 1220
deblock_luma_intra[1]_c 7314 4128
deblock_luma_intra[1]_neon 2163 793
---
common/arm/deblock-a.S | 209 ++++++++++++++++++++++++++++++++++++++++++++++++
common/deblock.c | 4 -
2 files changed, 209 insertions(+), 4 deletions(-)
diff --git a/common/arm/deblock-a.S b/common/arm/deblock-a.S
index 5a386a5..061ae49 100644
--- a/common/arm/deblock-a.S
+++ b/common/arm/deblock-a.S
@@ -195,6 +195,215 @@ function x264_deblock_h_luma_neon
bx lr
endfunc
+.macro h264_loop_filter_luma_intra
+ vdup.8 q14, r2 @ alpha
+ vdup.8 q15, r3 @ beta
+ vabd.u8 q4, q8, q0 @ abs(p0 - q0)
+ vabd.u8 q5, q9, q8 @ abs(p1 - p0)
+ vabd.u8 q6, q1, q0 @ abs(q1 - q0)
+ vclt.u8 q7, q4, q14 @ < alpha
+ vclt.u8 q5, q5, q15 @ < beta
+ vclt.u8 q6, q6, q15 @ < beta
+
+ vmov.u8 q13, #2
+ vshr.u8 q14, q14, #2 @ alpha >> 2
+ vadd.u8 q14, q14, q13 @ (alpha >> 2) + 2
+ vclt.u8 q13, q4, q14 @ < (alpha >> 2) + 2 if_2
+
+ vand q7, q7, q5
+ vand q12, q7, q6 @ if_1
+ vshrn.u16 d28, q12, #4
+ vrev64.32 d29, d28
+ vorr d28, d28, d29
+ vmov.32 r2, d28[0]
+ cmp r2, #0
+ beq 9f
+
+ sub sp, sp, #32
+ vst1.8 {q12-q13}, [sp,:128]
+
+ vshll.u8 q4, d18, #1 @ 2*p1
+ vshll.u8 q5, d19, #1
+ vaddw.u8 q4, q4, d16 @ 2*p1 + p0
+ vaddw.u8 q5, q5, d17
+ vaddw.u8 q4, q4, d2 @ 2*p1 + p0 + q1
+ vaddw.u8 q5, q5, d3
+ vrshrn.u16 d24, q4, #2
+ vrshrn.u16 d25, q5, #2
+
+ vaddl.u8 q6, d20, d16 @ p2 + p0
+ vaddl.u8 q7, d21, d17
+ vaddw.u8 q6, q6, d0 @ p2 + p0 + q0
+ vaddw.u8 q7, q7, d1
+ vadd.u16 q4, q4, q6 @ p2 + 2*p1 + 2*p0 + q0 + q1
+ vadd.u16 q5, q5, q7
+ vaddw.u8 q4, q4, d0 @ p2 + 2*p1 + 2*p0 + 2*q0 + q1
+ vaddw.u8 q5, q5, d1
+ vrshrn.u16 d26, q4, #3 @ p0'_2
+ vrshrn.u16 d27, q5, #3
+ vaddw.u8 q6, q6, d18 @ p2 + p1 + p0 + q0
+ vaddw.u8 q7, q7, d19
+ vrshrn.u16 d28, q6, #2 @ p1'_2
+ vrshrn.u16 d29, q7, #2
+ vaddl.u8 q4, d22, d20 @ p3 + p2
+ vaddl.u8 q5, d23, d21
+ vshl.u16 q4, q4, #1 @ 2*p3 + 2*p2
+ vshl.u16 q5, q5, #1
+ vadd.u16 q4, q4, q6 @ 2*p3 + 3*p2 + p1 + p0 + q0
+ vadd.u16 q5, q5, q7
+ vrshrn.u16 d30, q4, #3 @ p2'_2
+ vrshrn.u16 d31, q5, #3
+
+ vdup.8 q4, r3 @ beta
+ vabd.u8 q5, q10, q8 @ abs(p2 - p0)
+ vld1.8 {q6-q7}, [sp,:128] @ if_1, if_2
+ vclt.u8 q5, q5, q4 @ < beta if_3
+
+ vand q7, q7, q5 @ if_2 && if_3
+ vmvn q4, q7
+ vand q7, q7, q6 @ if_1 && if_2 && if_3
+ vand q6, q4, q6 @ if_1 && !(if_2 && if_3)
+
+ @ copy p0 to q15 so it can be clobbered
+ vbit q10, q15, q7
+ vmov q15, q8
+ vbit q8, q12, q6
+
+ @ wait for q9 to clobber
+ vshll.u8 q4, d2, #1 @ 2*q1
+ vshll.u8 q5, d3, #1
+
+ vbit q8, q12, q6
+
+ vaddw.u8 q4, q4, d0 @ 2*q1 + q0
+ vaddw.u8 q5, q5, d1
+
+ vbit q8, q13, q7
+
+ vaddw.u8 q4, q4, d18 @ 2*q1 + q0 + p1
+ vaddw.u8 q5, q5, d19
+
+ vbit q9, q14, q7
+
+ vrshrn.u16 d24, q4, #2
+ vrshrn.u16 d25, q5, #2
+
+ vaddl.u8 q6, d4, d0 @ q2 + q0
+ vaddl.u8 q7, d5, d1
+ vaddw.u8 q6, q6, d30 @ q2 + q0 + p0
+ vaddw.u8 q7, q7, d31
+ vadd.u16 q4, q4, q6 @ q2 + 2*q1 + 2*q0 + p0 + p1
+ vadd.u16 q5, q5, q7
+ vaddw.u8 q4, q4, d30 @ q2 + 2*q1 + 2*q0 + 2*p0 + p1
+ vaddw.u8 q5, q5, d31
+ vrshrn.u16 d26, q4, #3 @ q0'_2
+ vrshrn.u16 d27, q5, #3
+ vaddw.u8 q6, q6, d2 @ q2 + q1 + q0 + p0
+ vaddw.u8 q7, q7, d3
+ vrshrn.u16 d28, q6, #2 @ q1'_2
+ vrshrn.u16 d29, q7, #2
+ vaddl.u8 q4, d6, d4 @ q3 + q2
+ vaddl.u8 q5, d7, d5
+ vshl.u16 q4, q4, #1 @ 2*q3 + 2*q2
+ vshl.u16 q5, q5, #1
+ vadd.u16 q4, q4, q6 @ 2*q3 + 3*q2 + q1 + q0 + p0
+ vadd.u16 q5, q5, q7
+ vrshrn.u16 d30, q4, #3 @ q2'_2
+ vrshrn.u16 d31, q5, #3
+
+ vdup.8 q4, r3 @ beta
+ vabd.u8 q5, q2, q0 @ abs(q2 - q0)
+ vld1.8 {q6-q7}, [sp,:128]! @ if_1, if_2
+ vclt.u8 q5, q5, q4 @ < beta if_4
+
+ vand q7, q7, q5 @ if_2 && if_4
+ vmvn q4, q7
+ vand q7, q6, q7 @ if_1 && if_2 && if_4
+ vand q6, q6, q4 @ if_1 && !(if_2 && if_4)
+
+ vbit q0, q12, q6
+ vbit q1, q14, q7
+ vbit q0, q13, q7
+ vbit q2, q15, q7
+
+.endm
+
+function x264_deblock_v_luma_intra_neon
+ vld1.64 {d0, d1}, [r0,:128], r1
+ vld1.64 {d2, d3}, [r0,:128], r1
+ vld1.64 {d4, d5}, [r0,:128], r1
+ vld1.64 {d6, d7}, [r0,:128], r1
+ sub r0, r0, r1, lsl #3
+ vld1.64 {d22,d23}, [r0,:128], r1
+ vld1.64 {d20,d21}, [r0,:128], r1
+ vld1.64 {d18,d19}, [r0,:128], r1
+ vld1.64 {d16,d17}, [r0,:128]
+
+ align_push_regs
+
+ h264_loop_filter_luma_intra
+
+ sub r0, r0, r1, lsl #1
+ vst1.64 {d20,d21}, [r0,:128], r1
+ vst1.64 {d18,d19}, [r0,:128], r1
+ vst1.64 {d16,d17}, [r0,:128], r1
+ vst1.64 {d0, d1}, [r0,:128], r1
+ vst1.64 {d2, d3}, [r0,:128], r1
+ vst1.64 {d4, d5}, [r0,:128]
+9:
+ align_pop_regs
+ bx lr
+endfunc
+
+function x264_deblock_h_luma_intra_neon
+ sub r0, r0, #4
+ vld1.64 {d22}, [r0], r1
+ vld1.64 {d20}, [r0], r1
+ vld1.64 {d18}, [r0], r1
+ vld1.64 {d16}, [r0], r1
+ vld1.64 {d0}, [r0], r1
+ vld1.64 {d2}, [r0], r1
+ vld1.64 {d4}, [r0], r1
+ vld1.64 {d6}, [r0], r1
+ vld1.64 {d23}, [r0], r1
+ vld1.64 {d21}, [r0], r1
+ vld1.64 {d19}, [r0], r1
+ vld1.64 {d17}, [r0], r1
+ vld1.64 {d1}, [r0], r1
+ vld1.64 {d3}, [r0], r1
+ vld1.64 {d5}, [r0], r1
+ vld1.64 {d7}, [r0], r1
+
+ TRANSPOSE8x8 q11, q10, q9, q8, q0, q1, q2, q3
+
+ align_push_regs
+
+ h264_loop_filter_luma_intra
+
+ TRANSPOSE8x8 q11, q10, q9, q8, q0, q1, q2, q3
+
+ sub r0, r0, r1, lsl #4
+ vst1.64 {d22}, [r0], r1
+ vst1.64 {d20}, [r0], r1
+ vst1.64 {d18}, [r0], r1
+ vst1.64 {d16}, [r0], r1
+ vst1.64 {d0}, [r0], r1
+ vst1.64 {d2}, [r0], r1
+ vst1.64 {d4}, [r0], r1
+ vst1.64 {d6}, [r0], r1
+ vst1.64 {d23}, [r0], r1
+ vst1.64 {d21}, [r0], r1
+ vst1.64 {d19}, [r0], r1
+ vst1.64 {d17}, [r0], r1
+ vst1.64 {d1}, [r0], r1
+ vst1.64 {d3}, [r0], r1
+ vst1.64 {d5}, [r0], r1
+ vst1.64 {d7}, [r0], r1
+9:
+ align_pop_regs
+ bx lr
+endfunc
+
.macro h264_loop_filter_chroma
vdup.8 q11, r2 // alpha
vmovl.u8 q12, d24
diff --git a/common/deblock.c b/common/deblock.c
index 46379ec..3da99b0 100644
--- a/common/deblock.c
+++ b/common/deblock.c
@@ -745,11 +745,9 @@ void x264_deblock_h_chroma_intra_mbaff_neon( uint8_t *pix, intptr_t stride, int
void x264_deblock_h_chroma_intra_neon( uint8_t *pix, intptr_t stride, int alpha, int beta );
void x264_deblock_h_chroma_422_intra_neon( uint8_t *pix, intptr_t stride, int alpha, int beta );
void x264_deblock_v_chroma_intra_neon( uint8_t *pix, intptr_t stride, int alpha, int beta );
-#if ARCH_AARCH64
void x264_deblock_h_luma_intra_neon( uint8_t *pix, intptr_t stride, int alpha, int beta );
void x264_deblock_v_luma_intra_neon( uint8_t *pix, intptr_t stride, int alpha, int beta );
#endif
-#endif
#if !HIGH_BIT_DEPTH
#if HAVE_MSA
@@ -879,10 +877,8 @@ void x264_deblock_init( int cpu, x264_deblock_function_t *pf, int b_mbaff )
pf->deblock_h_chroma_420_intra = x264_deblock_h_chroma_intra_neon;
pf->deblock_h_chroma_422_intra = x264_deblock_h_chroma_422_intra_neon;
pf->deblock_chroma_intra[1] = x264_deblock_v_chroma_intra_neon;
-#if ARCH_AARCH64
pf->deblock_luma_intra[0] = x264_deblock_h_luma_intra_neon;
pf->deblock_luma_intra[1] = x264_deblock_v_luma_intra_neon;
-#endif
pf->deblock_strength = x264_deblock_strength_neon;
}
#endif
--
1.7.10.4
More information about the x264-devel
mailing list