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 <[email protected]> > --- > 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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev > _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
