[cfe-commits] r77556 - in /cfe/trunk: lib/AST/RecordLayoutBuilder.cpp lib/AST/RecordLayoutBuilder.h lib/CodeGen/CodeGenTypes.cpp test/CodeGenCXX/virt.cpp

[Douglas Gregor]

On Jul 30, 2009, at 8:30 AM, Anders Carlsson wrote:

>
> 30 jul 2009 kl. 08.28 skrev Douglas Gregor:
>>
>> At some point, I'm planning to kill off CXXRecordDecl entirely, so  
>> that we have one kind of RecordDecl that has all of the C and C++  
>> behavior in it. Of course, it will differentiate between C and C++  
>> records internally, to save space, but that difference won't be  
>> visible outside. This is one of those TODO items that keeps getting  
>> pushed down.
>>
>
> Yeah, that sounds like a great plan Until then, I think it's better  
> to use dyn_cast<CXXRecordDecl> instead of getLang().CPlusPlus  
> (although I'll admit to using the latter method at times too :)


I agree.

	- Doug