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

Uoti Urpala uoti.urpala
Wed Mar 5 07:25:51 CET 2008


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.





More information about the ffmpeg-devel mailing list