# HG changeset patch # User gpoirier # Date 1141590318 0 # Node ID ab1273ffe275113c504db12e175bb2cf3e3616ba # Parent 8b51e108cba6bcb05e8021e5604d1efd8f706d30 use xorps instead of mulps to toggle the sign of a float, as suggested by Software Optimization Guide for AMD64 Processors. Patch by Zuxy Meng < zuxy POIS meng AH gmail POIS com > OKed by Michael Original thread: Date: Mar 5, 2006 8:15 PM Subject: [Ffmpeg-devel] [PATCH] Little optimization to fft_sse.c diff -r 8b51e108cba6 -r ab1273ffe275 i386/fft_sse.c --- a/i386/fft_sse.c Sun Mar 05 19:02:35 2006 +0000 +++ b/i386/fft_sse.c Sun Mar 05 20:25:18 2006 +0000 @@ -23,14 +23,14 @@ #include -static const float p1p1p1m1[4] __attribute__((aligned(16))) = - { 1.0, 1.0, 1.0, -1.0 }; +static const int p1p1p1m1[4] __attribute__((aligned(16))) = + { 0, 0, 0, 1 << 31 }; -static const float p1p1m1p1[4] __attribute__((aligned(16))) = - { 1.0, 1.0, -1.0, 1.0 }; +static const int p1p1m1p1[4] __attribute__((aligned(16))) = + { 0, 0, 1 << 31, 0 }; -static const float p1p1m1m1[4] __attribute__((aligned(16))) = - { 1.0, 1.0, -1.0, -1.0 }; +static const int p1p1m1m1[4] __attribute__((aligned(16))) = + { 0, 0, 1 << 31, 1 << 31 }; #if 0 static void print_v4sf(const char *str, __m128 a) @@ -58,7 +58,6 @@ r = (__m128 *)&z[0]; c1 = *(__m128 *)p1p1m1m1; - c2 = *(__m128 *)p1p1p1m1; if (s->inverse) c2 = *(__m128 *)p1p1m1p1; else @@ -68,19 +67,20 @@ do { a = r[0]; b = _mm_shuffle_ps(a, a, _MM_SHUFFLE(1, 0, 3, 2)); - a = _mm_mul_ps(a, c1); + a = _mm_xor_ps(a, c1); /* do the pass 0 butterfly */ a = _mm_add_ps(a, b); a1 = r[1]; b = _mm_shuffle_ps(a1, a1, _MM_SHUFFLE(1, 0, 3, 2)); - a1 = _mm_mul_ps(a1, c1); + a1 = _mm_xor_ps(a1, c1); /* do the pass 0 butterfly */ b = _mm_add_ps(a1, b); /* multiply third by -i */ + /* by toggling the sign bit */ b = _mm_shuffle_ps(b, b, _MM_SHUFFLE(2, 3, 1, 0)); - b = _mm_mul_ps(b, c2); + b = _mm_xor_ps(b, c2); /* do the pass 1 butterfly */ r[0] = _mm_add_ps(a, b);