On Jan 21, 2010, at 11:22 AM, Ted Kremenek wrote:

> Author: kremenek
> Date: Thu Jan 21 13:22:34 2010
> New Revision: 94090
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=94090&view=rev
> Log:
> Allocate the 'Protocols' array in ObjCInterfaceType and
> ObjCObjectPointerType using the allocator associated with ASTContext.
> Not only does this fix a memory leak, but it also makes these arrays
> BumpPtrAllocated (in the typical case).

Why not allocate space for the protocols after the ObjCInterfaceType storage, 
rather than having

> +  ObjCProtocolDecl **Protocols;

?

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

Reply via email to