[FFmpeg-cvslog] x86: yadif: Mark mmxext optimizations as such

Diego Biurrun git at videolan.org
Tue Jul 31 23:05:10 CEST 2012


ffmpeg | branch: master | Diego Biurrun <diego at biurrun.de> | Tue Jul 31 12:17:02 2012 +0200| [d1505db06799d625420ff75de40af106c54e8c66] | committer: Diego Biurrun

x86: yadif: Mark mmxext optimizations as such

The yadif mmx optimizations contain the pmaxsw and pmaxub mmxext
instructions, causing sigills on CPUs that do not support mmxext.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d1505db06799d625420ff75de40af106c54e8c66
---

 libavfilter/x86/yadif.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/libavfilter/x86/yadif.c b/libavfilter/x86/yadif.c
index 72b06cc..beb0824 100644
--- a/libavfilter/x86/yadif.c
+++ b/libavfilter/x86/yadif.c
@@ -45,9 +45,9 @@ DECLARE_ASM_CONST(16, const xmm_reg, pw_1) = {0x0001000100010001ULL, 0x000100010
 #undef COMPILE_TEMPLATE_SSE
 #endif
 
-#if HAVE_MMX
+#if HAVE_MMX2
 #undef RENAME
-#define RENAME(a) a ## _mmx
+#define RENAME(a) a ## _mmx2
 #include "yadif_template.c"
 #endif
 
@@ -58,9 +58,9 @@ av_cold void ff_yadif_init_x86(YADIFContext *yadif)
     int cpu_flags = av_get_cpu_flags();
 
 #if HAVE_INLINE_ASM
-#if HAVE_MMX
-    if (cpu_flags & AV_CPU_FLAG_MMX)
-        yadif->filter_line = yadif_filter_line_mmx;
+#if HAVE_MMX2
+    if (cpu_flags & AV_CPU_FLAG_MMX2)
+        yadif->filter_line = yadif_filter_line_mmx2;
 #endif
 #if HAVE_SSE
     if (cpu_flags & AV_CPU_FLAG_SSE2)



More information about the ffmpeg-cvslog mailing list