On Mar 10, 2012, at 9:44 AM, David Blaikie wrote:
> This fixes a hole in handling arrays of arrays of user defined types
> that lead to the udt's ctor not being marked used (& not being
> generated in the case of a non trivial implicit dtor in a template) &
> not being access checked (in the case of a private dtor). The original
> functionality doesn't seem to have ever been tested - so if there's
> other test cases anyone thinks this functionality needs, I'd be happy
> to add those here too.

Looks good to me.

-            cast<CXXRecordDecl>(AllocType->getAs<RecordType>()->getDecl()))) {
+            
cast<CXXRecordDecl>(BaseAllocType->getAs<RecordType>()->getDecl()))) {

I know this is pre-existing, but please change the dominating isRecordType() 
check to use getAs and then re-use that result here.

John.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to