e5b335d367
Godot supports many different compilers and for production releases we have to support 3 currently: GCC8, Clang6, and MSVC2017. These compilers all do slightly different things with -ffast-math and it is causing issues now. See #24841, #24540, #10758, #10070. And probably other complaints about physics differences between release and release_debug builds. I've done some performance comparisons on Linux x86_64. All tests are ran 20 times. Bunnymark: (higher is better) (bunnies) min max stdev average fast-math 7332 7597 71 7432 this pr 7379 7779 108 7621 (102%) FPBench (gdscript port http://fpbench.org/) (lower is better) (ms) fast-math 15441 16127 192 15764 this pr 15671 16855 326 16001 (99%) Float_add (adding floats in a tight loop) (lower is better) (sec) fast-math 5.49 5.78 0.07 5.65 this pr 5.65 5.90 0.06 5.76 (98%) Float_div (dividing floats in a tight loop) (lower is better) (sec) fast-math 11.70 12.36 0.18 11.99 this pr 11.92 12.32 0.12 12.12 (99%) Float_mul (multiplying floats in a tight loop) (lower is better) (sec) fast-math 11.72 12.17 0.12 11.93 this pr 12.01 12.62 0.17 12.26 (97%) I have also looked at FPS numbers for tps-demo, 3d platformer, 2d platformer, and sponza and could not find any measurable difference. I believe that given the issues and oft-reported (physics) glitches on release builds I believe that the couple of percent of tight-loop floating point performance regression is well worth it. This fixes #24540 and fixes #24841
318 lines
11 KiB
Python
318 lines
11 KiB
Python
import os
|
|
import platform
|
|
import sys
|
|
from compat import decode_utf8
|
|
|
|
|
|
def is_active():
|
|
return True
|
|
|
|
|
|
def get_name():
|
|
return "X11"
|
|
|
|
|
|
def can_build():
|
|
|
|
if (os.name != "posix" or sys.platform == "darwin"):
|
|
return False
|
|
|
|
# Check the minimal dependencies
|
|
x11_error = os.system("pkg-config --version > /dev/null")
|
|
if (x11_error):
|
|
print("pkg-config not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config x11 --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("X11 not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config xcursor --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("xcursor not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config xinerama --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("xinerama not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config xrandr --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("xrandr not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config xrender --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("xrender not found.. x11 disabled.")
|
|
return False
|
|
|
|
x11_error = os.system("pkg-config xi --modversion > /dev/null ")
|
|
if (x11_error):
|
|
print("xi not found.. Aborting.")
|
|
return False
|
|
|
|
return True
|
|
|
|
def get_opts():
|
|
from SCons.Variables import BoolVariable, EnumVariable
|
|
|
|
return [
|
|
BoolVariable('use_llvm', 'Use the LLVM compiler', False),
|
|
BoolVariable('use_static_cpp', 'Link libgcc and libstdc++ statically for better portability', False),
|
|
BoolVariable('use_sanitizer', 'Use LLVM compiler address sanitizer', False),
|
|
BoolVariable('use_leak_sanitizer', 'Use LLVM compiler memory leaks sanitizer (implies use_sanitizer)', False),
|
|
BoolVariable('pulseaudio', 'Detect & use pulseaudio', True),
|
|
BoolVariable('udev', 'Use udev for gamepad connection callbacks', False),
|
|
EnumVariable('debug_symbols', 'Add debugging symbols to release builds', 'yes', ('yes', 'no', 'full')),
|
|
BoolVariable('separate_debug_symbols', 'Create a separate file containing debugging symbols', False),
|
|
BoolVariable('touch', 'Enable touch events', True),
|
|
BoolVariable('execinfo', 'Use libexecinfo on systems where glibc is not available', False),
|
|
]
|
|
|
|
|
|
def get_flags():
|
|
|
|
return [
|
|
('builtin_freetype', False),
|
|
('builtin_libpng', False),
|
|
('builtin_zlib', False),
|
|
]
|
|
|
|
|
|
def configure(env):
|
|
|
|
## Build type
|
|
|
|
if (env["target"] == "release"):
|
|
if (env["optimize"] == "speed"): #optimize for speed (default)
|
|
env.Prepend(CCFLAGS=['-O3'])
|
|
else: #optimize for size
|
|
env.Prepend(CCFLAGS=['-Os'])
|
|
|
|
if (env["debug_symbols"] == "yes"):
|
|
env.Prepend(CCFLAGS=['-g1'])
|
|
if (env["debug_symbols"] == "full"):
|
|
env.Prepend(CCFLAGS=['-g2'])
|
|
|
|
elif (env["target"] == "release_debug"):
|
|
if (env["optimize"] == "speed"): #optimize for speed (default)
|
|
env.Prepend(CCFLAGS=['-O2', '-DDEBUG_ENABLED'])
|
|
else: #optimize for size
|
|
env.Prepend(CCFLAGS=['-Os', '-DDEBUG_ENABLED'])
|
|
|
|
if (env["debug_symbols"] == "yes"):
|
|
env.Prepend(CCFLAGS=['-g1'])
|
|
if (env["debug_symbols"] == "full"):
|
|
env.Prepend(CCFLAGS=['-g2'])
|
|
|
|
elif (env["target"] == "debug"):
|
|
env.Prepend(CCFLAGS=['-g3', '-DDEBUG_ENABLED', '-DDEBUG_MEMORY_ENABLED'])
|
|
env.Append(LINKFLAGS=['-rdynamic'])
|
|
|
|
## Architecture
|
|
|
|
is64 = sys.maxsize > 2**32
|
|
if (env["bits"] == "default"):
|
|
env["bits"] = "64" if is64 else "32"
|
|
|
|
## Compiler configuration
|
|
|
|
if 'CXX' in env and 'clang' in os.path.basename(env['CXX']):
|
|
# Convenience check to enforce the use_llvm overrides when CXX is clang(++)
|
|
env['use_llvm'] = True
|
|
|
|
if env['use_llvm']:
|
|
if ('clang++' not in os.path.basename(env['CXX'])):
|
|
env["CC"] = "clang"
|
|
env["CXX"] = "clang++"
|
|
env["LINK"] = "clang++"
|
|
env.Append(CPPFLAGS=['-DTYPED_METHOD_BIND'])
|
|
env.extra_suffix = ".llvm" + env.extra_suffix
|
|
|
|
# leak sanitizer requires (address) sanitizer
|
|
if env['use_sanitizer'] or env['use_leak_sanitizer']:
|
|
env.Append(CCFLAGS=['-fsanitize=address', '-fno-omit-frame-pointer'])
|
|
env.Append(LINKFLAGS=['-fsanitize=address'])
|
|
env.extra_suffix += "s"
|
|
if env['use_leak_sanitizer']:
|
|
env.Append(CCFLAGS=['-fsanitize=leak'])
|
|
env.Append(LINKFLAGS=['-fsanitize=leak'])
|
|
|
|
if env['use_lto']:
|
|
env.Append(CCFLAGS=['-flto'])
|
|
if not env['use_llvm'] and env.GetOption("num_jobs") > 1:
|
|
env.Append(LINKFLAGS=['-flto=' + str(env.GetOption("num_jobs"))])
|
|
else:
|
|
env.Append(LINKFLAGS=['-flto'])
|
|
if not env['use_llvm']:
|
|
env['RANLIB'] = 'gcc-ranlib'
|
|
env['AR'] = 'gcc-ar'
|
|
|
|
env.Append(CCFLAGS=['-pipe'])
|
|
env.Append(LINKFLAGS=['-pipe'])
|
|
|
|
# Check for gcc version > 5 before adding -no-pie
|
|
import re
|
|
import subprocess
|
|
proc = subprocess.Popen([env['CXX'], '--version'], stdout=subprocess.PIPE)
|
|
(stdout, _) = proc.communicate()
|
|
stdout = decode_utf8(stdout)
|
|
match = re.search('[0-9][0-9.]*', stdout)
|
|
if match is not None:
|
|
version = match.group().split('.')
|
|
if (version[0] > '5'):
|
|
env.Append(CCFLAGS=['-fpie'])
|
|
env.Append(LINKFLAGS=['-no-pie'])
|
|
|
|
## Dependencies
|
|
|
|
env.ParseConfig('pkg-config x11 --cflags --libs')
|
|
env.ParseConfig('pkg-config xcursor --cflags --libs')
|
|
env.ParseConfig('pkg-config xinerama --cflags --libs')
|
|
env.ParseConfig('pkg-config xrandr --cflags --libs')
|
|
env.ParseConfig('pkg-config xrender --cflags --libs')
|
|
env.ParseConfig('pkg-config xi --cflags --libs')
|
|
|
|
if (env['touch']):
|
|
env.Append(CPPFLAGS=['-DTOUCH_ENABLED'])
|
|
|
|
# FIXME: Check for existence of the libs before parsing their flags with pkg-config
|
|
|
|
# freetype depends on libpng and zlib, so bundling one of them while keeping others
|
|
# as shared libraries leads to weird issues
|
|
if env['builtin_freetype'] or env['builtin_libpng'] or env['builtin_zlib']:
|
|
env['builtin_freetype'] = True
|
|
env['builtin_libpng'] = True
|
|
env['builtin_zlib'] = True
|
|
|
|
if not env['builtin_freetype']:
|
|
env.ParseConfig('pkg-config freetype2 --cflags --libs')
|
|
|
|
if not env['builtin_libpng']:
|
|
env.ParseConfig('pkg-config libpng --cflags --libs')
|
|
|
|
if not env['builtin_bullet']:
|
|
# We need at least version 2.88
|
|
import subprocess
|
|
bullet_version = subprocess.check_output(['pkg-config', 'bullet', '--modversion']).strip()
|
|
if bullet_version < "2.88":
|
|
# Abort as system bullet was requested but too old
|
|
print("Bullet: System version {0} does not match minimal requirements ({1}). Aborting.".format(bullet_version, "2.88"))
|
|
sys.exit(255)
|
|
env.ParseConfig('pkg-config bullet --cflags --libs')
|
|
|
|
if not env['builtin_enet']:
|
|
env.ParseConfig('pkg-config libenet --cflags --libs')
|
|
|
|
if not env['builtin_squish'] and env['tools']:
|
|
env.ParseConfig('pkg-config libsquish --cflags --libs')
|
|
|
|
if not env['builtin_zstd']:
|
|
env.ParseConfig('pkg-config libzstd --cflags --libs')
|
|
|
|
# Sound and video libraries
|
|
# Keep the order as it triggers chained dependencies (ogg needed by others, etc.)
|
|
|
|
if not env['builtin_libtheora']:
|
|
env['builtin_libogg'] = False # Needed to link against system libtheora
|
|
env['builtin_libvorbis'] = False # Needed to link against system libtheora
|
|
env.ParseConfig('pkg-config theora theoradec --cflags --libs')
|
|
else:
|
|
list_of_x86 = ['x86_64', 'x86', 'i386', 'i586']
|
|
if any(platform.machine() in s for s in list_of_x86):
|
|
env["x86_libtheora_opt_gcc"] = True
|
|
|
|
if not env['builtin_libvpx']:
|
|
env.ParseConfig('pkg-config vpx --cflags --libs')
|
|
|
|
if not env['builtin_libvorbis']:
|
|
env['builtin_libogg'] = False # Needed to link against system libvorbis
|
|
env.ParseConfig('pkg-config vorbis vorbisfile --cflags --libs')
|
|
|
|
if not env['builtin_opus']:
|
|
env['builtin_libogg'] = False # Needed to link against system opus
|
|
env.ParseConfig('pkg-config opus opusfile --cflags --libs')
|
|
|
|
if not env['builtin_libogg']:
|
|
env.ParseConfig('pkg-config ogg --cflags --libs')
|
|
|
|
if not env['builtin_libwebp']:
|
|
env.ParseConfig('pkg-config libwebp --cflags --libs')
|
|
|
|
if not env['builtin_mbedtls']:
|
|
# mbedTLS does not provide a pkgconfig config yet. See https://github.com/ARMmbed/mbedtls/issues/228
|
|
env.Append(LIBS=['mbedtls', 'mbedcrypto', 'mbedx509'])
|
|
|
|
if not env['builtin_libwebsockets']:
|
|
env.ParseConfig('pkg-config libwebsockets --cflags --libs')
|
|
|
|
if not env['builtin_miniupnpc']:
|
|
# No pkgconfig file so far, hardcode default paths.
|
|
env.Append(CPPPATH=["/usr/include/miniupnpc"])
|
|
env.Append(LIBS=["miniupnpc"])
|
|
|
|
# On Linux wchar_t should be 32-bits
|
|
# 16-bit library shouldn't be required due to compiler optimisations
|
|
if not env['builtin_pcre2']:
|
|
env.ParseConfig('pkg-config libpcre2-32 --cflags --libs')
|
|
|
|
## Flags
|
|
|
|
if (os.system("pkg-config --exists alsa") == 0): # 0 means found
|
|
print("Enabling ALSA")
|
|
env.Append(CPPFLAGS=["-DALSA_ENABLED", "-DALSAMIDI_ENABLED"])
|
|
# Don't parse --cflags, we don't need to add /usr/include/alsa to include path
|
|
env.ParseConfig('pkg-config alsa --libs')
|
|
else:
|
|
print("ALSA libraries not found, disabling driver")
|
|
|
|
if env['pulseaudio']:
|
|
if (os.system("pkg-config --exists libpulse") == 0): # 0 means found
|
|
print("Enabling PulseAudio")
|
|
env.Append(CPPFLAGS=["-DPULSEAUDIO_ENABLED"])
|
|
env.ParseConfig('pkg-config --cflags --libs libpulse')
|
|
else:
|
|
print("PulseAudio development libraries not found, disabling driver")
|
|
|
|
if (platform.system() == "Linux"):
|
|
env.Append(CPPFLAGS=["-DJOYDEV_ENABLED"])
|
|
|
|
if env['udev']:
|
|
if (os.system("pkg-config --exists libudev") == 0): # 0 means found
|
|
print("Enabling udev support")
|
|
env.Append(CPPFLAGS=["-DUDEV_ENABLED"])
|
|
env.ParseConfig('pkg-config libudev --cflags --libs')
|
|
else:
|
|
print("libudev development libraries not found, disabling udev support")
|
|
|
|
# Linkflags below this line should typically stay the last ones
|
|
if not env['builtin_zlib']:
|
|
env.ParseConfig('pkg-config zlib --cflags --libs')
|
|
|
|
env.Append(CPPPATH=['#platform/x11'])
|
|
env.Append(CPPFLAGS=['-DX11_ENABLED', '-DUNIX_ENABLED', '-DOPENGL_ENABLED', '-DGLES_ENABLED'])
|
|
env.Append(LIBS=['GL', 'pthread'])
|
|
|
|
if (platform.system() == "Linux"):
|
|
env.Append(LIBS=['dl'])
|
|
|
|
if (platform.system().find("BSD") >= 0):
|
|
env["execinfo"] = True
|
|
|
|
if env["execinfo"]:
|
|
env.Append(LIBS=['execinfo'])
|
|
|
|
## Cross-compilation
|
|
|
|
if (is64 and env["bits"] == "32"):
|
|
env.Append(CPPFLAGS=['-m32'])
|
|
env.Append(LINKFLAGS=['-m32', '-L/usr/lib/i386-linux-gnu'])
|
|
elif (not is64 and env["bits"] == "64"):
|
|
env.Append(CPPFLAGS=['-m64'])
|
|
env.Append(LINKFLAGS=['-m64', '-L/usr/lib/i686-linux-gnu'])
|
|
|
|
# Link those statically for portability
|
|
if env['use_static_cpp']:
|
|
env.Append(LINKFLAGS=['-static-libgcc', '-static-libstdc++'])
|