Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 11:12 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 10:18 AM, Hendrik Leppkes wrote: >> On Thu, Oct 15, 2015 at 4:04 PM, Derek Buitenhuis >> wrote: >>> On 10/15/2015 1:18 PM, Ganesh Ajjanagadde wrote: It has been demonstrated that using libc provided floati

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 10:18 AM, Hendrik Leppkes wrote: > On Thu, Oct 15, 2015 at 4:04 PM, Derek Buitenhuis > wrote: >> On 10/15/2015 1:18 PM, Ganesh Ajjanagadde wrote: >>> It has been demonstrated that using libc provided floating point >>> functions is beneficial, in the context of fabs() vs F

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Hendrik Leppkes
On Thu, Oct 15, 2015 at 4:04 PM, Derek Buitenhuis wrote: > On 10/15/2015 1:18 PM, Ganesh Ajjanagadde wrote: >> It has been demonstrated that using libc provided floating point >> functions is beneficial, in the context of fabs() vs FFABS. >> >> Unfortunately, MSVC 2012 (and earlier) lack the ISO C

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Derek Buitenhuis
On 10/15/2015 1:18 PM, Ganesh Ajjanagadde wrote: > It has been demonstrated that using libc provided floating point > functions is beneficial, in the context of fabs() vs FFABS. > > Unfortunately, MSVC 2012 (and earlier) lack the ISO C99 fmax, fmaxf, > fmin, fminf functions. This patch adds them,

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Clément Bœsch
On Thu, Oct 15, 2015 at 09:10:00AM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:31 AM, Clément Bœsch wrote: > > On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: > >> It has been demonstrated that using libc provided floating point > >> functions is beneficial, in

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde mit.edu> writes: > > Ganesh Ajjanagadde gmail.com> writes: > > > >> It has been demonstrated that using libc provided floating point > >> functions is beneficial, in the context of fabs() vs FFABS. > > > > Please provide actual numbers for this patch. > > For a change, why do

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ronald S. Bultje
Hi, On Thu, Oct 15, 2015 at 9:10 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:31 AM, Clément Bœsch wrote: > > On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: > >> It has been demonstrated that using libc provided floating point > >> functions is beneficial, in t

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 9:10 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:31 AM, Clément Bœsch wrote: >> On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: >>> It has been demonstrated that using libc provided floating point >>> functions is beneficial, in the contex

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 8:20 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde gmail.com> writes: > >> It has been demonstrated that using libc provided floating point >> functions is beneficial, in the context of fabs() vs FFABS. > > Please provide actual numbers for this patch. For a change, wh

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 8:31 AM, Clément Bœsch wrote: > On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: >> It has been demonstrated that using libc provided floating point >> functions is beneficial, in the context of fabs() vs FFABS. >> >> Unfortunately, MSVC 2012 (and earlier

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Clément Bœsch
On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: > It has been demonstrated that using libc provided floating point > functions is beneficial, in the context of fabs() vs FFABS. > > Unfortunately, MSVC 2012 (and earlier) lack the ISO C99 fmax, fmaxf, > fmin, fminf functions. Thi

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde gmail.com> writes: > It has been demonstrated that using libc provided floating point > functions is beneficial, in the context of fabs() vs FFABS. Please provide actual numbers for this patch. Carl Eugen ___ ffmpeg-devel mailing l

[FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Ganesh Ajjanagadde
It has been demonstrated that using libc provided floating point functions is beneficial, in the context of fabs() vs FFABS. Unfortunately, MSVC 2012 (and earlier) lack the ISO C99 fmax, fmaxf, fmin, fminf functions. This patch adds them, thus making their usage in FFmpeg safe. Signed-off-by: Gan