There's an entry in the changelog: "Improved decoding efficiency of all bit depths but especially so for 24 bits (lvqcl)" A couple of comments: 1) The patch that improves encoding for all depths was proposed by Miroslav Lichvar <http://git.xiph.org/?p=flac.git;a=commit;h=4eab6313cd2198b5647d925bdb3847590505fa21> 2) "Performance checks" graph posted by Martijn van Beurden reminded me that the decoding improvements for FLAC were made only for 32-bit x64 architecture (aka IA32). FLAC is multiplatform, so it make sense to mention this.
On Wed, Nov 26, 2014 at 06:09:38PM +0300, lvqcl wrote:> There's an entry in the changelog: > > "Improved decoding efficiency of all bit depths but especially so for 24 bits (lvqcl)" > > A couple of comments: > > 1) The patch that improves encoding for all depths was proposed by Miroslav Lichvar > <http://git.xiph.org/?p=flac.git;a=commit;h=4eab6313cd2198b5647d925bdb3847590505fa21> > > > 2) "Performance checks" graph posted by Martijn van Beurden reminded me that > the decoding improvements for FLAC were made only for 32-bit x64 architecture > (aka IA32). FLAC is multiplatform, so it make sense to mention this.IIRC for other archs most of the decoding speed improvement was already in 1.3.0. That's where the C version of the read_rice_signed_block function was optimized. Now in 1.3.1 it's enabled also for x86, but the difference between the x86 assembly and the C function is much smaller. -- Miroslav Lichvar
lvqcl wrote:> There's an entry in the changelog: > > "Improved decoding efficiency of all bit depths but especially so for 24 bits (lvqcl)" > > A couple of comments: > > 1) The patch that improves encoding for all depths was proposed by Miroslav Lichvar > <http://git.xiph.org/?p=flac.git;a=commit;h=4eab6313cd2198b5647d925bdb3847590505fa21> > > > 2) "Performance checks" graph posted by Martijn van Beurden reminded me that > the decoding improvements for FLAC were made only for 32-bit x64 architecture > (aka IA32). FLAC is multiplatform, so it make sense to mention this.Thanks. Updated to add Miroslav Lichvar and mention IA32 archirecture. Erik -- ---------------------------------------------------------------------- Erik de Castro Lopo http://www.mega-nerd.com/