[Ffmpeg-cvslog] r8474 - trunk/libavcodec/resample2.c

[Michael Niedermayer]

Hi

On Fri, Mar 23, 2007 at 01:02:38PM +0100, mkhodor6 at yahoo.com wrote:
> While you're arguing over what gcc does or doesn't optimize here, may
> I point out the foolishness of
> 
>     for(i=1; i<50; i++){
> 
> for the calculation of a Maclaurin series, the rate of convergence of
> which is dependent on the input value?  What you probably want is
> 
>     while(t > some_threshold)

as the input value is known to be small, 50 is plenty. of course a threshold
based check could be used too which might but doesnt have to be faster,
as a integer comparission is replaced by a float comparission also a
compiler can unroll a i<50 loop much easier then a t > T loop

anyway patch with benchmark object file size and precission comparision is
welcome


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070323/c6e8ff83/attachment.pgp>