From ab9cda1497e7bed41b474a0ab8a2a4b0547dd916 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Verschelde?= Date: Thu, 14 Dec 2017 17:59:18 +0100 Subject: [PATCH] openmp: Don't try to link GCC's implementation on Clang Also disable openmp build on Travis, breaks on Trusty's Clang. Group deps more naturally. --- .travis.yml | 14 +++++++------- platform/x11/detect.py | 5 +++-- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/.travis.yml b/.travis.yml index c11a21aeef3..0dfeaf16e1a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -48,14 +48,14 @@ addons: - pkg-config - libx11-dev - libxcursor-dev - - libasound2-dev - - libfreetype6-dev - - libgl1-mesa-dev - - libglu1-mesa-dev - - libssl-dev + - libxi-dev - libxinerama-dev - libxrandr-dev - - libxi-dev + - libgl1-mesa-dev + - libglu1-mesa-dev + - libasound2-dev + - libfreetype6-dev + - libssl-dev # For cross-compiling to Windows. #- binutils-mingw-w64-i686 @@ -90,5 +90,5 @@ script: - if [ "$STATIC_CHECKS" = "yes" ]; then sh ./misc/travis/clang-format.sh; else - scons -j2 CC=$CC CXX=$CXX platform=$GODOT_TARGET TOOLS=$TOOLS verbose=yes progress=no; + scons -j2 CC=$CC CXX=$CXX platform=$GODOT_TARGET TOOLS=$TOOLS verbose=yes progress=no openmp=no; fi diff --git a/platform/x11/detect.py b/platform/x11/detect.py index 0f07e22e6b6..ed75e5eab57 100644 --- a/platform/x11/detect.py +++ b/platform/x11/detect.py @@ -265,9 +265,10 @@ def configure(env): env.Append(LINKFLAGS=['-m64', '-L/usr/lib/i686-linux-gnu']) - if (env["openmp"]): + if env["openmp"]: env.Append(CPPFLAGS=['-fopenmp']) - env.Append(LIBS=['gomp']) + if not env['use_llvm']: + env.Append(LIBS=['gomp']) if env['use_static_cpp']: env.Append(LINKFLAGS=['-static-libstdc++'])