[FFmpeg-cvslog] r16674 - trunk/configure

Måns Rullgård mans
Thu Jan 22 10:24:03 CET 2009


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.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-cvslog mailing list