search for: stream_encoder_intrin_sse2

...with the 32-bit accumulator and correctly selects the 64-bit accumulator with very large 16-bit partitions. This is related to commits 6f7ec60c and 187e596e. --- src/libFLAC/include/private/stream_encoder.h | 6 ++++++ src/libFLAC/stream_encoder.c | 14 ++++++-------- src/libFLAC/stream_encoder_intrin_sse2.c | 3 ++- src/libFLAC/stream_encoder_intrin_ssse3.c | 3 ++- 4 files changed, 16 insertions(+), 10 deletions(-) diff --git a/src/libFLAC/include/private/stream_encoder.h b/src/libFLAC/include/private/stream_encoder.h index d26039a..8147f9e 100644 --- a/src/libFLAC/include/private/stream_...

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...

Anybody still has MSVC 2015 update 1? It seems that MSVC 2015U2 miscompiles stream_encoder_intrin_sse2.c, stream_encoder_intrin_ssse3.c (and probably stream_encoder_intrin_avx2.c, but currently I cannot test it 'cause I don't have avx2 CPU). I wonder what happened and what changed in the asm code that MSVC generates.

Erik de Castro Lopo <mle+la at mega-nerd.com> wrote: >> -msse for SSE code, -msse2 for SSE2 code, -msse4.1 for SSE4.1 code > > Yes, that was it. Brain fart. These flags were not needed on x86_64. > > Erik But now all C code is compiled with -msse2 and it won't work on older CPUs. Isn't it better to compile only necessary files with this flag?

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 :