[FFmpeg-devel] [PATCH] move ff_emulated_edge_mc() to dsputil

Diego Biurrun diego
Wed Mar 5 08:44:24 CET 2008


On Wed, Mar 05, 2008 at 08:25:51AM +0200, Uoti Urpala wrote:
> On Wed, 2008-03-05 at 01:04 -0500, Rich Felker wrote:
> > On Wed, Mar 05, 2008 at 07:32:07AM +0200, Uoti Urpala wrote:
> > > On Tue, 2008-03-04 at 23:51 -0500, Rich Felker wrote:
> > > > Splitting dsputil.c and motion_est.c would be a major improvement in
> > > > compilability. These files are waaaaaay too big and gcc stupidity
> > > > results in very slow and memory-intensive compiling, probably due to
> > > > useless searches for global optimizations or something.
> > > 
> > > In the case of motion_est.c I'd blame programmer stupidity more than
> > > compiler stupidity. It has big av_always_inline functions and macros
> > > using them which are then used lots of times. The code could be written
> > > more efficiently. (That's the same issue I mentioned earlier when
> > > discussing object sizes.)
> > 
> > Patch welcome.</michaelni>
> 
> http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2008-January/041456.html
> 
> Btw changing "av_always_inline int cmp" to "av_noinline int cmp" and
> comparing object sizes before and after should give you some idea of the
> amount of repeating code in that file.

Michael, did you miss that patch during your review runs?

Diego




More information about the ffmpeg-devel mailing list