Re: [FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-14 Thread Ronald S. Bultje
Hi, On Mon, Mar 14, 2016 at 10:06 PM, Ganesh Ajjanagadde wrote: > On Mon, Mar 14, 2016 at 8:56 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Sun, Mar 13, 2016 at 12:34 PM, Ganesh Ajjanagadde > > > wrote: > >> > >> On Sun, Mar 13,

Re: [FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-14 Thread Ganesh Ajjanagadde
On Mon, Mar 14, 2016 at 8:56 AM, Ronald S. Bultje wrote: > Hi, > > On Sun, Mar 13, 2016 at 12:34 PM, Ganesh Ajjanagadde > wrote: >> >> On Sun, Mar 13, 2016 at 7:51 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Sat, Mar 12, 2016

Re: [FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-14 Thread Ronald S. Bultje
Hi, On Sun, Mar 13, 2016 at 12:34 PM, Ganesh Ajjanagadde wrote: > On Sun, Mar 13, 2016 at 7:51 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Sat, Mar 12, 2016 at 11:40 AM, Ganesh Ajjanagadde > > > wrote: > >> > >> diff --git

Re: [FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-13 Thread Ganesh Ajjanagadde
On Sun, Mar 13, 2016 at 7:51 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Mar 12, 2016 at 11:40 AM, Ganesh Ajjanagadde > wrote: >> >> diff --git a/libavutil/internal.h b/libavutil/internal.h >> index da76ca2..aa43754 100644 >> --- a/libavutil/internal.h

Re: [FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-13 Thread Ronald S. Bultje
Hi, On Sat, Mar 12, 2016 at 11:40 AM, Ganesh Ajjanagadde wrote: > diff --git a/libavutil/internal.h b/libavutil/internal.h > index da76ca2..aa43754 100644 > --- a/libavutil/internal.h > +++ b/libavutil/internal.h > @@ -315,6 +315,22 @@ static av_always_inline float

[FFmpeg-devel] [PATCHv2] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

2016-03-12 Thread Ganesh Ajjanagadde
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