Skip to content
Snippets Groups Projects
Commit 496bcbb0 authored by Ronald S. Bultje's avatar Ronald S. Bultje
Browse files

Fix overflow in temporal_filter_apply_sse2().

The accumulator array is an integer array, so use paddd instead of paddw
to add values to it. Fixes overflows when using large --arnr-maxframes
(>8) values.

Change-Id: Iad83794caa02400a65f3ab5760f2517e082d66ae
parent 73c3d327
No related branches found
No related tags found
No related merge requests found
...@@ -164,10 +164,10 @@ temporal_filter_apply_load_finished: ...@@ -164,10 +164,10 @@ temporal_filter_apply_load_finished:
movdqa xmm6, [rdi+32] movdqa xmm6, [rdi+32]
movdqa xmm7, [rdi+48] movdqa xmm7, [rdi+48]
; += modifier ; += modifier
paddw xmm4, xmm0 paddd xmm4, xmm0
paddw xmm5, xmm2 paddd xmm5, xmm2
paddw xmm6, xmm1 paddd xmm6, xmm1
paddw xmm7, xmm3 paddd xmm7, xmm3
; write back ; write back
movdqa [rdi], xmm4 movdqa [rdi], xmm4
movdqa [rdi+16], xmm5 movdqa [rdi+16], xmm5
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment