search for: stream_encoder_intrin_

Further refactoring/simplifying. The 1st patch moves a code that calculates abs. value into a separate function. The 2nd patch combines two intrinsic instructions into one line of code. -------------- next part -------------- A non-text attachment was scrubbed... Name: stream_enc_1.patch Type: application/octet-stream Size: 3672 bytes Desc: not available Url :

lvqcl wrote: > Further refactoring/simplifying. Both applied. Thanks. Erik -- ---------------------------------------------------------------------- Erik de Castro Lopo http://www.mega-nerd.com/

Erik de Castro Lopo wrote: > Both applied. Thanks. Thanks! But for some reason stream_encoder_intrin_sse2.c wasn't updated by the 2nd patch...

Erik de Castro Lopo wrote: > Hmm. I remember that the second patch didn't apply cleanly > after the first patch and just assumed that the second patch > duplicated changes in the first patch. > > Please provide another patch and we will try again. The diff to the current git is attached. > PS: I've also been accepting pull requests via github if you'd > like to

As I wrote earlier, MSVC 2015 U2 incorrectly compiles stream_encoder_intrin_*.c files for x86-64 platform. As a result, flac works, but compression ratio is close to 1. This patch disables some compiler optimizations, and compression ratio reverts back to normal values. -------------- next part -------------- A non-text attachment was scrubbed... Name: msvc_bug.patch Type:...

Erik de Castro Lopo wrote: >> As I wrote earlier, MSVC 2015 U2 incorrectly compiles >> stream_encoder_intrin_*.c files for x86-64 platform. >> As a result, flac works, but compression ratio is close to 1. >> This patch disables some compiler optimizations, and >> compression ratio reverts back to normal values. > > Rather than having the same chunk of code in three different > fi...