Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Muhammad Faiz
On Tue, Nov 3, 2015 at 1:20 PM, Ganesh Ajjanagadde wrote: > On Tue, Nov 3, 2015 at 12:54 PM, Muhammad Faiz wrote: >> On Tue, Nov 3, 2015 at 3:47 AM, Ganesh Ajjanagadde wrote: >>> On Tue, Nov 3, 2015 at 5:06 AM, Muhammad Faiz wrote: On Sat, Oct 31, 2015 at 10:15 AM, Ganesh Ajjanagadde >>>

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Ganesh Ajjanagadde
On Tue, Nov 3, 2015 at 12:54 PM, Muhammad Faiz wrote: > On Tue, Nov 3, 2015 at 3:47 AM, Ganesh Ajjanagadde wrote: >> On Tue, Nov 3, 2015 at 5:06 AM, Muhammad Faiz wrote: >>> On Sat, Oct 31, 2015 at 10:15 AM, Ganesh Ajjanagadde >>> wrote: On Fri, Oct 30, 2015 at 7:29 PM, Michael Niedermaye

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Muhammad Faiz
On Tue, Nov 3, 2015 at 9:21 AM, Hendrik Leppkes wrote: > On Tue, Nov 3, 2015 at 5:10 PM, Muhammad Faiz wrote: >> On Tue, Nov 3, 2015 at 5:42 AM, Clément Bœsch wrote: >>> On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Le

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Muhammad Faiz
On Tue, Nov 3, 2015 at 3:47 AM, Ganesh Ajjanagadde wrote: > On Tue, Nov 3, 2015 at 5:06 AM, Muhammad Faiz wrote: >> On Sat, Oct 31, 2015 at 10:15 AM, Ganesh Ajjanagadde >> wrote: >>> On Fri, Oct 30, 2015 at 7:29 PM, Michael Niedermayer >>> wrote: On Fri, Oct 30, 2015 at 06:53:34PM -0400,

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Hendrik Leppkes
On Tue, Nov 3, 2015 at 5:10 PM, Muhammad Faiz wrote: > On Tue, Nov 3, 2015 at 5:42 AM, Clément Bœsch wrote: >> On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: >>> On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: >>> > On Fri, Oct 30, 2015 at 11:35 PM, Michael

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Ganesh Ajjanagadde
On Tue, Nov 3, 2015 at 11:10 AM, Muhammad Faiz wrote: > On Tue, Nov 3, 2015 at 5:42 AM, Clément Bœsch wrote: >> On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: >>> On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: >>> > On Fri, Oct 30, 2015 at 11:35 PM, Michael

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Muhammad Faiz
On Tue, Nov 3, 2015 at 5:42 AM, Clément Bœsch wrote: > On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: >> On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: >> > On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer >> > wrote: >> > > From: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Clément Bœsch
On Tue, Nov 03, 2015 at 02:42:55PM +0100, Clément Bœsch wrote: [...] > > > Wasn't there a concern that these emulations don't behave identical to > > > the C library versions in regards to NaN handling? > > > I faintly remember something in the previous discussion. > > > > yes, i forgot about that

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Clément Bœsch
On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: > On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: > > On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer > > wrote: > > > From: Michael Niedermayer > > > > > > This should fix the build failure of avf_showcq

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Ganesh Ajjanagadde
On Tue, Nov 3, 2015 at 5:06 AM, Muhammad Faiz wrote: > On Sat, Oct 31, 2015 at 10:15 AM, Ganesh Ajjanagadde wrote: >> On Fri, Oct 30, 2015 at 7:29 PM, Michael Niedermayer >> wrote: >>> On Fri, Oct 30, 2015 at 06:53:34PM -0400, Ganesh Ajjanagadde wrote: On Fri, Oct 30, 2015 at 6:35 PM, Micha

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Muhammad Faiz
On Sat, Oct 31, 2015 at 10:15 AM, Ganesh Ajjanagadde wrote: > On Fri, Oct 30, 2015 at 7:29 PM, Michael Niedermayer > wrote: >> On Fri, Oct 30, 2015 at 06:53:34PM -0400, Ganesh Ajjanagadde wrote: >>> On Fri, Oct 30, 2015 at 6:35 PM, Michael Niedermayer >>> wrote: >>> > From: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Ganesh Ajjanagadde
On Fri, Oct 30, 2015 at 7:29 PM, Michael Niedermayer wrote: > On Fri, Oct 30, 2015 at 06:53:34PM -0400, Ganesh Ajjanagadde wrote: >> On Fri, Oct 30, 2015 at 6:35 PM, Michael Niedermayer >> wrote: >> > From: Michael Niedermayer >> > >> > This should fix the build failure of avf_showcqt.c >> > >>

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Ganesh Ajjanagadde
On Fri, Oct 30, 2015 at 7:33 PM, Michael Niedermayer wrote: > On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: >> On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer >> wrote: >> > From: Michael Niedermayer >> > >> > This should fix the build failure of avf_showcqt.c >> > >> >

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Michael Niedermayer
On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: > On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer > wrote: > > From: Michael Niedermayer > > > > This should fix the build failure of avf_showcqt.c > > > > An alternative solution would be to add a check for fmin/fmax to > >

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Michael Niedermayer
On Fri, Oct 30, 2015 at 06:53:34PM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 30, 2015 at 6:35 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > This should fix the build failure of avf_showcqt.c > > > > An alternative solution would be to add a check for fmin/fmax to > >

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Hendrik Leppkes
On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > This should fix the build failure of avf_showcqt.c > > An alternative solution would be to add a check for fmin/fmax to fate-source > and > then to replace them by FFMIN/FFMAX, i can do that if preferred

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Ganesh Ajjanagadde
On Fri, Oct 30, 2015 at 6:35 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > This should fix the build failure of avf_showcqt.c > > An alternative solution would be to add a check for fmin/fmax to fate-source > and > then to replace them by FFMIN/FFMAX, i can do that if preferred?

[FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-10-30 Thread Michael Niedermayer
From: Michael Niedermayer This should fix the build failure of avf_showcqt.c An alternative solution would be to add a check for fmin/fmax to fate-source and then to replace them by FFMIN/FFMAX, i can do that if preferred? Untested due to lack of a affected platform Signed-off-by: Michael Nied