[Lldb-commits] [lldb] r151198 - /lldb/trunk/Makefile

[Benjamin Kramer]

On 22.02.2012, at 22:30, Greg Clayton wrote:

> Benjamin, 
> 
> For performance reasons we don't want RTTI enabled in LLVM and Clang. It is not that we can't enable it, it is really what the llvm and clang folks want so that llvm and clang perform well.

I agree, I was trying to work around a bug in g++ but that didn't work. With this change everything is back to normal (no rtti anywhere) again. Sorry for the noise.

- Ben

> Greg
> 
> On Feb 22, 2012, at 1:00 PM, Benjamin Kramer wrote:
> 
>> Author: d0k
>> Date: Wed Feb 22 15:00:17 2012
>> New Revision: 151198
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=151198&view=rev
>> Log:
>> Revert the RTTI change from r151187. It make lldb compile with g++ 4.4 but it doesn't link anymore. The bug seems to be fixed in g++ 4.5.
>> 
>> Modified:
>>   lldb/trunk/Makefile
>> 
>> Modified: lldb/trunk/Makefile
>> URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/Makefile?rev=151198&r1=151197&r2=151198&view=diff
>> ==============================================================================
>> --- lldb/trunk/Makefile (original)
>> +++ lldb/trunk/Makefile Wed Feb 22 15:00:17 2012
>> @@ -25,9 +25,6 @@
>> # Set LLVM source root level.
>> LEVEL := $(LLDB_LEVEL)/../..
>> 
>> -# shared_ptr from libstdc++ requires RTTI.
>> -REQUIRES_RTTI := 1
>> -
>> # Include LLVM common makefile.
>> include $(LEVEL)/Makefile.common
>> 
>> 
>> 
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>