[flac-dev] exhaustive-model-search issue results in multi-gigabyte FLAC file

Erik de Castro Lopo mle+la at mega-nerd.com
Wed Jul 17 03:23:20 PDT 2013


Miroslav Lichvar wrote:

> On Wed, Jul 17, 2013 at 07:45:53PM +1000, Erik de Castro Lopo wrote:
> > The fix was changing one local variable from FLAC_uint32 to FLAC_uint64
> > in function precompute_partition_info_sums_().
> > 
> >     https://git.xiph.org/?p=flac.git;a=commit;h=6f7ec60c7e7f05f5ab0b1cf6b7b0945e44afcd4b
> 
> I don't like this fix. It will probably hurt performance with 16-bit
> data and it won't fix the problem in the assembly code.
> 
> I think the check if 32-bit accumulator is enough should be improved
> instead if possible.

I'm currently working on a test for this issue. Once we have a test
in the test suite I would be happy to accept patches that improve
performance.

I like to make it correct before I make it fast.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/


More information about the flac-dev mailing list