[Mesa-dev] [PATCH] scons: Add -fno-rtti to CXXFLAGS with llvm-3.2.

[Jose Fonseca]

Looks good to me.

I hope this doesn't affect any other C++ code we might interface with, but I don't think that's a problem ATM.

Jose

----- Original Message -----
> llvm-3.2svn r166772 no longer requires RTTI for lib/Support.
> 
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  scons/llvm.py | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/scons/llvm.py b/scons/llvm.py
> index f87766a..e1ed760 100644
> --- a/scons/llvm.py
> +++ b/scons/llvm.py
> @@ -183,6 +183,9 @@ 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:
> --
> 1.7.12.1
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>