[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