[FFmpeg-devel] MMX version for put_no_rnd_h264_chroma_mc8_c

Guillaume POIRIER poirierg
Sat Dec 22 00:12:08 CET 2007


Hello,

On Dec 21, 2007 7:38 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>
> On Wed, Dec 05, 2007 at 03:41:02PM +0100, Christophe GISQUET wrote:
> > Christophe GISQUET a ?crit :
> > > Good evening,
> > >
> > > Michael Niedermayer a ?crit :
> > >>> Before:
> > >>> VC-1: 2085 dezicycles in rnd, 1047692 runs, 884 skips
> > >>> h264: 1093 dezicycles in rnd, 2096936 runs, 216 skips
> > >>>
> > >>> Patch applied:
> > >>> VC-1: 2106 dezicycles in rnd, 1047537 runs, 1039 skips
> > >>>       2119 dezicycles in no_rnd, 1047384 runs, 1192 skips
> > >>> h264: 1097 dezicycles in rnd, 2096867 runs, 285 skips
> > >> i assume these measurements where over the function calls not were
> > >> the *_TIMER were placed in the last patch?
> > >
> > > For the version with my patch, I simply put them around call to
> > > put_h264_chroma_mc8_mmx in put_h264_chroma_mc8_mmx_rnd and
> > > put_h264_chroma_mc8_mmx_nornd
> > >
> > > For the "before" timing, I created an intermediate function mimicking
> > > the same thing. Save for difference in inlining treatment, this should
> > > be equivalent and comparable to timing with patch applied.
> > >
> > > The global measures are without the _TIMER and the intermediate
> > > function, running with -benchmark.
> >
> > Ping ?
>
> i thought ive ok-ed it ...
> if it was unclear ...
> patch looks ok
>


Patch applied.

Guillaume
-- 
A soldier will fight long and hard for a bit of colored ribbon.
 -- Napoleon Bonaparte




More information about the ffmpeg-devel mailing list