Hello, Few comments.
You can use VBROADCASTI128 macro instead of changing the size of the constants (VBROADCASTI128 load 128 bit when using XMM, and broadcast the 128bit to the two lane when using YMM) The %if ARCH_X86_64 part, seems strange. seems to only be useful for AVX2, not for sse/avx. Not directly related to this patch, but it can be interesting for testing purpose to write a checkasm test for the v210 func decoding. So it's more easy to check the perf for "each" cpu flags, and be sure, the various width cases works as expected. Martin _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
