[FFmpeg-cvslog] r16674 - trunk/configure

[Diego Biurrun]

On Wed, Jan 21, 2009 at 10:22:00PM -0500, Alexander Strange wrote:
> 
> On Jan 21, 2009, at 7:35 PM, M?ns Rullg?rd wrote:
> 
> > Alexander Strange <astrange at ithinksw.com> writes:
> >
> >> On Jan 18, 2009, at 2:38 PM, M?ns Rullg?rd wrote:
> >>
> >>> Diego Biurrun <diego at biurrun.de> writes:
> >>>
> >>>> On Sun, Jan 18, 2009 at 07:29:57PM +0100, mru wrote:
> >>>>>
> >>>>> Log:
> >>>>> configure: add fastdiv to CONFIG_LIST
> >>>>
> >>>> This is missing from the help output.
> >>>
> >>> You keep saying that...
> >>
> >> The option has no effect on x86/arm anyway, since the CONFIG_FASTDIV
> >> check in libavutil/internal.h is only an #elif.
> >> I'd rather remove CONFIG_FASTDIV; obviously nobody cares about it,
> >> since it has never been defined since fastdiv was added.
> >> It might be nice to remove ff_inverse under CONFIG_SMALL, though.
> >
> > It was benchmarked and found to be faster on every architecture that
> > was tested.
> 
> Yes, I meant removing CONFIG_FASTDIV and leaving it always enabled.

This will have the extra benefit of fixing the help output.
I'm all for it.

Diego