[FFmpeg-devel] C99 or GCC extensions

Axel Holzinger aholzinger
Sat Jul 12 09:58:20 CEST 2008


Michael Niedermayer <michaelni at gmx.at> writes:
> On Fri, Jul 11, 2008 at 08:02:28PM +0200, Axel Holzinger wrote:
> > Axel Holzinger <aholzinger at gmx.de> writes:
> >
> > > "M?ns Rullg?rd" <mans at mansr.com> writes:
> > > > It is customary to write "static type" rather than "type
> > > static".
> > >
> > > Will do so.
> > >
> > > > Lose the backslashes.
> > >
> > > Ditto.
> > >
> > > > >  #elif defined(CONFIG_FASTDIV)
> > > > >  #    define FASTDIV(a,b)
> > > > ((uint32_t)((((uint64_t)a)*ff_inverse[b])>>32))
> > > > >  #else
> > > >
> > > > Unrelated to this change, machine-specific code should go into
> > > > subdirs, IMHO.
> >
> > Okay, didn't get that. Yes it would be better to put the machine-
> > specific part into subdirs. But in libavutil there aren't yet any
> > subdirs, so we would start doing so. I will study how this is
> > done in lavf or lavc, but don't be too impatient, I'm still
> > learning :-)
> >
> > So here's version two with the promised changes.
> 
> Please also post some benchmarks that show that the uses of FASTDIV
> didnt become slower. (see START/STOP_TIMER)

Will do so, although I won't be able to do this for ARM V4, which is
also concerned.

Of course I checked the disassembly on Intel C compiler and I couldn't
see a difference in the code.

Cheers
Axel





More information about the ffmpeg-devel mailing list