On 10/21/17, Martin Vignali <martin.vign...@gmail.com> wrote: > Hello, > > In attach patch to add AVX2 version for add_bytes > > 0001-libavcodec-lossless_videodsp-add-add_bytes-avx2-vers : > add AVX2 version > > pass fate-test for me (os 10.12, x86_64) > > checkasm result : (Kaby Lake) (run 10 times, and i took the fastest > version) > checkasm: all 2 tests passed > add_bytes_c: 108.7 > add_bytes_sse2: 26.5 > add_bytes_avx2: 15.5 > > > 0002-libavcodec-lossless_video_dsp-cosmetic-add-better-se: > only cosmetic > like the ref c function declaration in asm file is not consistent between > each asm file > i think a better separator for each function make the file easier to read > > also add the c declaration for add bytes in comment > > > Martin >
Are you sure 32bit alignment is actually enforced? _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel