[FFmpeg-devel] [PATCH] avfilter/vsrc_mandelbrot: change sin to sinf for color computation

2015-11-24 Thread Ganesh Ajjanagadde
lrintf is anyway used, suggesting we only care up to floating precision. Rurthermore, there is a compat hack in avutil/libm for this function, and it is used in avcodec/aacps_tablegen.h. This yields a non-negligible speedup. Sample benchmark: x86-64, Haswell, GNU/Linux: old (draw_mandelbrot):

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_mandelbrot: change sin to sinf for color computation

2015-11-24 Thread Michael Niedermayer
On Tue, Nov 24, 2015 at 01:05:20PM -0500, Ganesh Ajjanagadde wrote: > lrintf is anyway used, suggesting we only care up to floating precision. > Rurthermore, there is a compat hack in avutil/libm for this function, > and it is used in avcodec/aacps_tablegen.h. > > This yields a non-negligible

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_mandelbrot: change sin to sinf for color computation

2015-11-24 Thread Ganesh Ajjanagadde
On Tue, Nov 24, 2015 at 2:28 PM, Michael Niedermayer wrote: > On Tue, Nov 24, 2015 at 01:05:20PM -0500, Ganesh Ajjanagadde wrote: >> lrintf is anyway used, suggesting we only care up to floating precision. >> Rurthermore, there is a compat hack in avutil/libm for this function,