lavc/aacenc_utils: replace powf(x,y) by expf(logf(x), y)
authorGanesh Ajjanagadde <gajjanag@gmail.com>
Tue, 8 Mar 2016 02:16:29 +0000 (21:16 -0500)
committerGanesh Ajjanagadde <gajjanag@gmail.com>
Fri, 18 Mar 2016 14:47:25 +0000 (07:47 -0700)
commitbccc81dfa08e6561df6ed37860e3a08f7d983825
tree0671f9c1f2d6c6849eaf278a53b531a0b35b05e5
parentd6e76dd13239829a62db3b83d54163d797654bf9
lavc/aacenc_utils: replace powf(x,y) by expf(logf(x), y)

This is ~2x faster for y not an integer on Haswell+GCC, and should
generally be faster due to the fact that anyway powf essentially does
this under the hood. Made an inline function in lavu/internal.h for this
purpose.

Note that there are some accuracy differences, that should generally be
negligible. In particular, FATE still passes on this platform.

Results in ~ 7% speedup in aac encoding with -march=native, Haswell+GCC.
before:
ffmpeg -i sin.flac -acodec aac -y sin_new.aac  6.05s user 0.06s system 104% cpu 5.821 total

after:
ffmpeg -i sin.flac -acodec aac -y sin_new.aac  5.67s user 0.03s system 105% cpu 5.416 total

This is also faster than an alternative approach that pulls in powf, gets rid of
the crufty NaN checks and other special cases, exploits knowledge about the intervals, etc.
This of course does not exclude smarter approaches; just suggests that
there would need to be significant work on this front of lower utility than
searches for hotspots elsewhere.

Reviewed-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
Reviewed-by: Ronald S. Bultje <rsbultje@gmail.com>
Signed-off-by: Ganesh Ajjanagadde <gajjanag@gmail.com>
libavcodec/aacenc_utils.h
libavutil/internal.h