From ce8eadb6e8adc24f675b364e0620dbf1c9e079a8 Mon Sep 17 00:00:00 2001 From: Alexander von Gluck IV Date: Sat, 12 Oct 2013 17:12:31 +0000 Subject: build: remove forced -fno-rtti * As discussed on the mailing list, forced no-rtti breaks C++ public API's such as the Haiku C++ libGL.so * -fno-rtti *can* be still set however instead of blindly forcing -fno-rtti, we can rely on the llvm-config --cppflags output. If the system llvm is built without rtti (default), the no-rtti flag will be present in llvm-config --cppflags (which we pick up on) If llvm is built with rtti (REQUIRES_RTTI=1), then -fno-rtti is removed from llvm-config --cppflags. * We could selectively add / remove rtti from various components, however mixing rtti and non-rtti code is tricky and could introduce missing symbols. * This needs impact tested. Reviewed-by: Francisco Jerez --- scons/llvm.py | 3 --- 1 file changed, 3 deletions(-) (limited to 'scons/llvm.py') diff --git a/scons/llvm.py b/scons/llvm.py index 7cd609c2095..c1c37362d3c 100644 --- a/scons/llvm.py +++ b/scons/llvm.py @@ -195,9 +195,6 @@ def generate(env): if llvm_version >= distutils.version.LooseVersion('3.1'): components.append('mcjit') - if llvm_version >= distutils.version.LooseVersion('3.2'): - env.Append(CXXFLAGS = ('-fno-rtti',)) - env.ParseConfig('llvm-config --libs ' + ' '.join(components)) env.ParseConfig('llvm-config --ldflags') except OSError: -- cgit v1.2.3