[FFmpeg-cvslog] h264: change deblock_h_chroma_8_mmxext() to prevent valgrind confusion.

Michael Niedermayer michaelni at gmx.at
Fri Feb 17 22:49:47 CET 2012


On Fri, Feb 17, 2012 at 10:44:45PM +0100, Reimar Döffinger wrote:
> On 17 Feb 2012, at 21:57, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Fri, Feb 17, 2012 at 09:56:30PM +0100, Reimar Döffinger wrote:
> >> On 17 Feb 2012, at 21:41, git at videolan.org (Michael Niedermayer) wrote:
> >> 
> >>> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Fri Feb 17 20:41:33 2012 +0100| [f9caec0cf9a2bd06efa3bc800649713e8531cad0] | committer: Michael Niedermayer
> >>> 
> >>> h264: change deblock_h_chroma_8_mmxext() to prevent valgrind confusion.
> >> 
> >> I had guessed this was to reduce cache pressure. Otherwise there isn't much reason to use call in the other case either? (didn't check too carefully though).
> > 
> > better solutions are welcome
> 
> I was mostly wondering if I was missing something.

Well, i didnt benchmark it so the new code may be faster or slower
it has fewer instructions though and its just about 50 instructions
that get inlined so i dont think it can have any major cache effect.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20120217/9cbccaf3/attachment.asc>


More information about the ffmpeg-cvslog mailing list