[FFmpeg-cvslog] r16674 - trunk/configure

[Diego Biurrun]

On Thu, Jan 22, 2009 at 09:24:03AM +0000, M?ns Rullg?rd wrote:
> Alexander Strange <astrange at ithinksw.com> writes:
> 
> > 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.
> 
> That puts us back at the question whether or not to tie it to
> CONFIG_SMALL.

Why is that an issue?  Just enable it always, then decide what to do in
the small case..

Diego