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

Douglas Gregor dgregor at apple.com
Thu Jul 30 08:31:49 PDT 2009

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

More information about the cfe-commits mailing list