Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

configure.ac: fix AVX, SSE and MMX options #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ffontaine
Copy link

@ffontaine ffontaine commented Nov 22, 2020

AVX, SSE and MMX options are broken since 87a900c

For example, when the user enables SSE, it will also enable MMX and the user can't disable MMX

Signed-off-by: Fabrice Fontaine fontaine.fabrice@gmail.com

AVX, SSE and MMX options are broken since
freeswitch@87a900c

For example, when the user enables SSE, it will also enable MMX and the
user can't disable MMX

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@coppice-git
Copy link
Collaborator

coppice-git commented Nov 23, 2020

The cascading of higher levels of MMX/SSE/AVX enabling the lower levels is by design. There are no devices with SSE which do not support MMX.

If spandsp is not building for an i686 machine, this needs to be fixed. What is the build platform where it is failing?

I just tried building the current spandsp code on an (not very recent) Fedora i686 machine with "./configure --enable-sse2" and it builds OK. So, the reported problem seems to relate to a particular environment.

@ffontaine
Copy link
Author

Build failure is raised on buildroot by the following toolchain:
http://autobuild.buildroot.org/toolchains/tarballs/br-i386-pentium4-full-2020.02.tar.bz2
I opened issue #21

@xrobau
Copy link

xrobau commented May 14, 2021

The build script in #21 is explicitly enabling things that the CPU target doesn't support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants