Compile fails when ARCH_HAVE_SSE about opus HOT 6 CLOSED

patch working for me: https://gist.github.com/snorkellingcactus/c2d2209822bbdb4038c797232b127de1

build log: https://gist.github.com/snorkellingcactus/c4a629cd6c34c979c0696bdacdb8b2a7

from opus.

I can reproduce by disabling the OPUS_CHECK_INTRINSICS([SSE2],...) call around configure.ac line 559. The tests just assume SSE2 is available.

I worry that your patch is just disabling the tests, rather than fixing the macros to use the right variant on your system.

Which gcc is this that doesn't support anything past SSE? The other side of our C89 compatibility commitment, I guess. :)

from opus.

The compiler supports sse and all typical stuff. But i'm using icecc for distributed compilation so i can't use -march=native directly. Instead i set the explicit C[XX]FLAGS that the compiler sends to the preprocessor when i pass -march=native. Because of this you can see some -mno-sseN and a bunch of C[XX]FLAGS.

The compiler is GCC 5.4.0.

I build opus on an Intel Celeron which have sse only. ¿isn't celt_inner_prod_sse the right expansion for it?.

I can try to compile it without icecc or custom C[XX]FLAGS and see if i can reproduce the error.

Thanks.

from opus.

I think it should work fine in master now, since this fix landed: https://git.xiph.org/?p=opus.git;a=commit;h=4507637c
Can you confirm it fixes the issue you're reporting?

from opus.

I repeated the reproduction test and the issue is resolved for me. Feel free to reopen if the there are still issues for your setup.

from opus.

Looks like it's solved. Thanks!

from opus.