Teeworlds: Build fails to build on non-x86 (because of immintrin.h)
../src/engine/client/client.cpp:5:10: fatal error: immintrin.h: No such file or directory
#include <immintrin.h> //_mm_pause
^~~~~~~~~~~~~
ignatenkobrain
All 5 comments
The fix was not ported to CMake, I believe.
On 64bit, you can simply comment out the #include <immintrin.h>.
If it doesn't work, can you try to add the compilation flag -msse2? It should fix it afaik.
1844
Dune-jr
on 15 Dec 2018
@Dune-jr I don't think that -msse2 will work on ppc64le or s390x or arm, because it is x86-specific.
ignatenkobrain
on 15 Dec 2018
Right.
Since you're on 64-bit, it should simply work without #include <immintrin.h>
Dune-jr
on 15 Dec 2018
@Dune-jr there is also 32bit arm ;)
ignatenkobrain
on 15 Dec 2018
Copied from https://github.com/teeworlds/teeworlds/issues/1844#issuecomment-446756342:
Note that this will still fail to compile for architectures other than Intel (and AMD) ones. The code in question is quite unportable:
I suggest to remove it until a portable solution is found. Teeworlds code (especially outside src/base/system.c) should not rely on such compiler- and architecture-specific functions without fallback.
To elaborate: There are CPU architectures besides Intel, and we shouldn't rely on a specific one. If we do, we should do it in src/base/system.c but not make compilation fail on unknown architectures.
heinrich5991
on 15 Dec 2018
Related issues
oy
·
5Comments
fokkonaut
·
5Comments
axblk
·
3Comments
Dune-jr
·
5Comments
AmonBune
·
4Comments
Most helpful comment
Copied from https://github.com/teeworlds/teeworlds/issues/1844#issuecomment-446756342:
To elaborate: There are CPU architectures besides Intel, and we shouldn't rely on a specific one. If we do, we should do it in src/base/system.c but not make compilation fail on unknown architectures.