[FFmpeg-cvslog] r26033 - trunk/libavcodec/ac3enc.c

Justin Ruggles justin.ruggles
Mon Dec 20 23:44:51 CET 2010


On 12/20/2010 05:33 PM, Ronald S. Bultje wrote:

> Hi,
> 
> On Mon, Dec 20, 2010 at 5:30 PM, Justin Ruggles
> <justin.ruggles at gmail.com> wrote:
>> All of the configurations which failed after this revision are x86.
> 
> Is it possible that one of the optimized functions on these platforms,
> which isn't tested otherwise, is causing this? E.g. consider that
> float-to-int routines require different float values depending on
> optimization level...

I tried all x86 versions of sad[0] (C, MMX, MMX2, SSE2) and they passed
on my system, so I don't think there is anything fundamentally wrong
with how ac3enc is using sad[0].

>> Unrelated to this (but still concerning FATE failures), the ARM and
>> AVR32 configurations did not like the previous ac3 encoder changes.  The
>> output size is the same, but the checksums differ.  My best guess is
>> that it is related to the lrintf() in the float-to-int conversion for
>> the sin/cos tables.  I could remove the lrintf() but I don't think it's
>> a good idea.  Alternatively, I could hardcode the tables like is done
>> for the MDCT window and that would probably fix it.
> 
> That's what the float off-by-one tests are for...


How does one enable this for an audio encoding regression test?

-Justin



More information about the ffmpeg-cvslog mailing list