[x265] patch0.patch doesn't work

[chen]

I guess this is SSSE3 intrinsic, we need modify cmake
ex: merge below two part into one "-mssse3"
 
            set_source_files_properties(
                vec/blockcopy-sse3.cpp vec/pixel-sse3.cpp vec/dct-sse3.cpp vec/intra-sse3.cpp
                PROPERTIES COMPILE_FLAGS "-msse3")
            set_source_files_properties(
                vec/ipfilter-ssse3.cpp vec/pixel-ssse3.cpp vec/dct-ssse3.cpp
                PROPERTIES COMPILE_FLAGS "-mssse3")




在 2013-10-21 04:44:00，"Steve Borho" <steve at borho.org> 写道：






On Sun, Oct 20, 2013 at 7:51 AM, JMK <three4teen16 at coldmail.nu> wrote:
--- chenm003 at 163.com wrote:

> a little typo problem, thanks
> here is temporary patch

Thanks for the attention, but the patch you've sent doesn't work...

[ 68%] Building CXX object common/CMakeFiles/common.dir/vec/intra-sse3.cpp.obj
In file included from F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp:32:0:
e:\gcc481\mingw\lib\gcc\i686-w64-mingw32\4.8.1\include\tmmintrin.h:31:3: error: #error "SSSE3 instruction set not enabled"
 # error "SSSE3 instruction set not enabled"
   ^
F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp: In function 'void {anonymous}::intra_pred_dc(pixel*, pixel*, pixel*, intptr_t, int, int)':
F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp:178:74: error: '_mm_hadd_epi16' was not declared in this scope
         sum  = _mm_cvtsi128_si32(_mm_hadd_epi16(_mm_hadd_epi16(temp, temp), temp));
                                                                          ^
F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp:178:81: error: '_mm_hadd_epi16' was not declared in this scope
         sum  = _mm_cvtsi128_si32(_mm_hadd_epi16(_mm_hadd_epi16(temp, temp), temp));
                                                                                 ^
F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp:178:81: error: redeclaration of '<typeprefixerror>_mm_hadd_epi16'
F:/GITS/X265/20131020/source/common/vec/intra-sse3.cpp:178:74: note: previous declaration '<typeprefixerror>_mm_hadd_epi16'
         sum  = _mm_cvtsi128_si32(_mm_hadd_epi16(_mm_hadd_epi16(temp, temp), temp));
                                                                          ^
make[2]: *** [common/CMakeFiles/common.dir/vec/intra-sse3.cpp.obj] Error 1
make[1]: *** [common/CMakeFiles/common.dir/all] Error 2
make: *** [all] Error 2



the new function is using SSE4.1 instructions, so the function needs to be moved to intra-sse41.cpp 


--
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131021/e5f0d055/attachment.html>