[FFmpeg-devel] [PATCH 2/4] tablegen: Don't use cbrtf in host tools

Michael Niedermayer michaelni at gmx.at
Wed Jan 16 01:44:06 CET 2013


On Tue, Jan 15, 2013 at 07:06:25PM -0500, Derek Buitenhuis wrote:
> On 15/01/2013 6:43 PM, Michael Niedermayer wrote:
> > is there a reason you dont use pow () ?
> > pow should be even more widespread than powf
> 
> I copied the cbrtf implementation from our own libm.h.
> 
> And it *is* cbrt*f*...

the code in libm.h is speed relevant as its used during runtime
while the table generator is used during buildtime to build a
relativly small table.
powf() is possibly faster than pow() so it makes sense to use it in
libm.h
while for the table generators the portability argument applies more

anyway this is just a suggestion i dont mind powf() being used if it
works on all supported platforms
    
[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

During times of universal deceit, telling the truth becomes a
revolutionary act. -- George Orwell
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130116/c819088a/attachment.asc>


More information about the ffmpeg-devel mailing list