[x265] [PATCH] sse.inc: Replace sse_ss24 vector class function with intrinsic
dnyaneshwar at multicorewareinc.com
dnyaneshwar at multicorewareinc.com
Wed Oct 9 13:51:39 CEST 2013
# HG changeset patch
# User Dnyaneshwar Gorade <dnyaneshwar at multicorewareinc.com>
# Date 1381319455 -19800
# Wed Oct 09 17:20:55 2013 +0530
# Node ID 17b76c50fee08abe92c93d1a1d5f2529a93c4bf9
# Parent 603af6e8fa6cc9ecca931408d16fe37dc667e356
sse.inc: Replace sse_ss24 vector class function with intrinsic.
diff -r 603af6e8fa6c -r 17b76c50fee0 source/common/vec/sse.inc
--- a/source/common/vec/sse.inc Wed Oct 09 17:13:50 2013 +0530
+++ b/source/common/vec/sse.inc Wed Oct 09 17:20:55 2013 +0530
@@ -195,43 +195,51 @@
{
int rows = ly;
- Vec4i diff(0);
- Vec8s m1, n1;
- Vec4i sum(0);
+ __m128i sum = _mm_setzero_si128();
+ __m128i m1, n1, diff, sign1, sign2, tmp1, tmp2;
+
+#define PROCESS_SSE_SS24x1(BASE)\
+ m1 = _mm_loadu_si128((__m128i const*)(fenc + BASE)); \
+ n1 = _mm_loadu_si128((__m128i const*)(fref + BASE)); \
+ sign1 = _mm_srai_epi16(m1, 15); \
+ tmp1 = _mm_unpacklo_epi16(m1, sign1); \
+ sign2 = _mm_srai_epi16(n1, 15); \
+ tmp2 = _mm_unpacklo_epi16(n1, sign2); \
+ diff = _mm_sub_epi32(tmp1, tmp2); \
+ diff = _mm_mullo_epi32(diff, diff); \
+ sum = _mm_add_epi32(sum, diff)
+
for (; rows != 0; rows--)
{
- m1.load(fenc);
- n1.load(fref);
- diff = extend_low(m1) - extend_low(n1);
- diff = diff * diff;
- sum += diff;
- diff = extend_high(m1) - extend_high(n1);
- diff = diff * diff;
- sum += diff;
+ PROCESS_SSE_SS24x1(0);
- m1.load(fenc + 8);
- n1.load(fref + 8);
- diff = extend_low(m1) - extend_low(n1);
- diff = diff * diff;
- sum += diff;
- diff = extend_high(m1) - extend_high(n1);
- diff = diff * diff;
- sum += diff;
+ m1 = _mm_unpackhi_epi16(m1, sign1);
+ n1 = _mm_unpackhi_epi16(n1, sign2);
+ diff = _mm_sub_epi32(m1, n1);
+ diff = _mm_mullo_epi32(diff, diff);
+ sum = _mm_add_epi32(sum, diff);
- m1.load(fenc + 16);
- n1.load(fref + 16);
- diff = extend_low(m1) - extend_low(n1);
- diff = diff * diff;
- sum += diff;
- diff = extend_high(m1) - extend_high(n1);
- diff = diff * diff;
- sum += diff;
+ PROCESS_SSE_SS24x1(8);
+
+ m1 = _mm_unpackhi_epi16(m1, sign1);
+ n1 = _mm_unpackhi_epi16(n1, sign2);
+ diff = _mm_sub_epi32(m1, n1);
+ diff = _mm_mullo_epi32(diff, diff);
+ sum = _mm_add_epi32(sum, diff);
+
+ PROCESS_SSE_SS24x1(16);
+
+ m1 = _mm_unpackhi_epi16(m1, sign1);
+ n1 = _mm_unpackhi_epi16(n1, sign2);
+ diff = _mm_sub_epi32(m1, n1);
+ diff = _mm_mullo_epi32(diff, diff);
+ sum = _mm_add_epi32(sum, diff);
fenc += strideFenc;
fref += strideFref;
}
-
- return horizontal_add(sum);
+ __m128i sum1 = _mm_hadd_epi32(sum, sum); // horizontally add 4 elements
+ return _mm_cvtsi128_si32(_mm_hadd_epi32(sum1, sum1));
}
template<int ly>
More information about the x265-devel
mailing list