[FFmpeg-cvslog] lavfi/nlmeans: fix aarch64 assembly with clang
Jan Ekström
git at videolan.org
Sat Jul 28 17:42:01 EEST 2018
ffmpeg | branch: master | Jan Ekström <jeebjp at gmail.com> | Thu Jul 26 23:45:01 2018 +0300| [eb94ec3257940c6d63462c2c228d42e366449a9f] | committer: Jan Ekström
lavfi/nlmeans: fix aarch64 assembly with clang
Clang is more strict about some things.
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=eb94ec3257940c6d63462c2c228d42e366449a9f
---
libavfilter/aarch64/vf_nlmeans_neon.S | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavfilter/aarch64/vf_nlmeans_neon.S b/libavfilter/aarch64/vf_nlmeans_neon.S
index 6308a428db..e69b0dd923 100644
--- a/libavfilter/aarch64/vf_nlmeans_neon.S
+++ b/libavfilter/aarch64/vf_nlmeans_neon.S
@@ -22,7 +22,7 @@
// acc_sum_store(ABCD) = {X+A, X+A+B, X+A+B+C, X+A+B+C+D}
.macro acc_sum_store x, xb
- dup v24.4S, v24.4S[3] // ...X -> XXXX
+ dup v24.4S, v24.S[3] // ...X -> XXXX
ext v25.16B, v26.16B, \xb, #12 // ext(0000,ABCD,12)=0ABC
add v24.4S, v24.4S, \x // XXXX+ABCD={X+A,X+B,X+C,X+D}
add v24.4S, v24.4S, v25.4S // {X+A,X+B+A,X+C+B,X+D+C} (+0ABC)
@@ -37,7 +37,7 @@ function ff_compute_safe_ssd_integral_image_neon, export=1
movi v26.4S, #0 // used as zero for the "rotations" in acc_sum_store
sub x3, x3, w6, UXTW // s1 padding (s1_linesize - w)
sub x5, x5, w6, UXTW // s2 padding (s2_linesize - w)
- sub x9, x0, x1, UXTW #2 // dst_top
+ sub x9, x0, w1, UXTW #2 // dst_top
sub x1, x1, w6, UXTW // dst padding (dst_linesize_32 - w)
lsl x1, x1, #2 // dst padding expressed in bytes
1: mov w10, w6 // width copy for each line
More information about the ffmpeg-cvslog
mailing list