[cfe-commits] r60947 - in /cfe/trunk: include/clang/AST/DeclBase.h lib/Sema/SemaDecl.cpp lib/Sema/SemaDeclCXX.cpp test/SemaCXX/constructor.cpp test/SemaCXX/destructor.cpp

[Douglas Gregor]

On Dec 13, 2008, at 4:15 PM, Sebastian Redl wrote:

> Douglas Gregor wrote:
>> Author: dgregor
>> Date: Fri Dec 12 02:25:50 2008
>> New Revision: 60947
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=60947&view=rev
>> Log:
>> Enable out-of-line definitions of C++ constructors and destructors
>>
> Something is wrong with this. It merges the declaration and the  
> definition of the out-of-line destructor. Then both the  
> CXXRecordDecl and the TranslationUnit think they own the constructor  
> decl and try to free it.


You're right! Thanks for the heads-up; I wasn't seeing the failure on  
Mac OS X. I fixed this with a subsequent patch.

	- Doug