[FFmpeg-devel] [PATCH] x86/swr: add ff_float_to_int32_a_avx2

Christophe Gisquet christophe.gisquet at gmail.com
Thu Nov 6 22:35:33 CET 2014


Hi,

2014-11-06 21:48 GMT+01:00 James Almer <jamrial at gmail.com>:
> 13797 decicycles in ff_float_to_int32_a_sse2, 32768 runs, 0 skips
> 8603 decicycles in ff_float_to_int32_a_avx2, 32766 runs, 2 skips

A couple of naïve questions (I haven't checked):
Does it increase the alignment requirement?
If yes, should it be notified somewhere (API bump, comment in the
relevant header, ...)?

> x86inc.asm doesn't seem to handle cmpps or its aliases properly when using avx.

I remember fixing its declaration (missing one parameter) while
working on aac (maybe one year ago). It's unrelated, maybe?

Otherwise looks obvious.

-- 
Christophe


More information about the ffmpeg-devel mailing list