search

jgaeddert / liquid-dsp

digital signal processing library for software-defined radios
http://liquidsdr.org
MIT License
1.87k stars 438 forks source link

cannot be compiled with SSE2 #137

Closed bmwiedemann closed 5 years ago

bmwiedemann commented 6 years ago

While working on reproducible builds for openSUSE, I found that liquid-dsp (commit 4c9a7514) cannot be compiled with SSE2

src/dotprod/src/dotprod_cccf.mmx.c line 49 includes and uses pmmintrin.h even when no SSE3 support is available or desired, leading to compilation errors

In file included from src/dotprod/src/dotprod_cccf.mmx.c:49:
/usr/lib64/gcc/x86_64-suse-linux/8/include/pmmintrin.h:50:1: error: inlining failed in call to always_inline '_mm_addsub_ps': target specific option mismatch
 _mm_addsub_ps (__m128 __X, __m128 __Y)
 ^~~~~~~~~~~~~
src/dotprod/src/dotprod_cccf.mmx.c:255:13: note: called from here
          s = _mm_addsub_ps( ci, cq );
bmwiedemann commented 5 years ago

ping. Can you reproduce this? Anything else I can help?

fsheikh commented 5 years ago

Yeah I could also see this with a force compilation for SSE2 on x86/Ubunut 16.04. Fix patch underway