x86: h264: Don't keep data in the redzone across function calls on 64 bit unix
authorMartin Storsjö <martin@martin.st>
Mon, 20 Feb 2012 09:24:35 +0000 (11:24 +0200)
committerMartin Storsjö <martin@martin.st>
Tue, 10 Jun 2014 13:31:48 +0000 (16:31 +0300)
commit570d4b21863b6254d6bbca9c528bede471bb4478
tree1a6bb2680e594b4ff09c665702393dbdc6d6afdb
parent0776e0ef6ba4160281ef3fabea43e670f3792b4a
x86: h264: Don't keep data in the redzone across function calls on 64 bit unix

We know that the called function (ff_chroma_inter_body_mmxext)
doesn't touch the redzone, and thus will be kept intact - thus,
this doesn't fix any bug per se.

However, valgrind's memcheck tool intentionally assumes that the
redzone is clobbered on every function call and function return
(see a long comment in valgrind/memcheck/mc_main.c). This avoids
false positives in that tool, at the cost of an extra stack pointer
adjustment.

The other alternative would be a valgrind suppression for this issue,
but that's an extra burden for everybody that wants to run libavcodec
within valgrind.

Signed-off-by: Martin Storsjö <martin@martin.st>
libavcodec/x86/h264_deblock.asm