[x265] [PATCH] blockcopy-sse3.cpp: removed unnecessary variable

dnyaneshwar at multicorewareinc.com dnyaneshwar at multicorewareinc.com
Fri Oct 18 12:45:03 CEST 2013


# HG changeset patch
# User Dnyaneshwar Gorade <dnyaneshwar at multicorewareinc.com>
# Date 1382092987 -19800
#      Fri Oct 18 16:13:07 2013 +0530
# Node ID 0c59d0a0ad6f72d9af436c8b9c1a2a3ff8ef1027
# Parent  f3523973eafba8b2157d87bdc06932bf8bac47bf
blockcopy-sse3.cpp: removed unnecessary variable.

diff -r f3523973eafb -r 0c59d0a0ad6f source/common/vec/blockcopy-sse3.cpp
--- a/source/common/vec/blockcopy-sse3.cpp	Fri Oct 18 14:18:05 2013 +0530
+++ b/source/common/vec/blockcopy-sse3.cpp	Fri Oct 18 16:13:07 2013 +0530
@@ -134,11 +134,10 @@
 void pixeladd_pp(int bx, int by, pixel *dst, intptr_t dstride, pixel *src0, pixel *src1, intptr_t sstride0, intptr_t sstride1)
 {
     size_t aligncheck = (size_t)dst | (size_t)src0 | bx | sstride0 | sstride1 | dstride;
-    unsigned char i = (1 << X265_DEPTH) - 1;
 
     if (!(aligncheck & 15))
     {
-        __m128i maxval = _mm_set1_epi8(i);
+        __m128i maxval = _mm_set1_epi8((unsigned char)((1 << X265_DEPTH) - 1));
         __m128i zero = _mm_setzero_si128();
 
         // fast path, multiples of 16 pixel wide blocks
@@ -162,7 +161,7 @@
     }
     else if (!(bx & 15))
     {
-        __m128i maxval = _mm_set1_epi8(i);
+        __m128i maxval = _mm_set1_epi8((unsigned char)((1 << X265_DEPTH) - 1));
         __m128i zero = _mm_setzero_si128();
 
         // fast path, multiples of 16 pixel wide blocks but pointers/strides require unaligned accesses


More information about the x265-devel mailing list