On Jan 21, 2010, at 3:29 PM, Ted Kremenek wrote:
> Yes would could do that. It would save a pointer.
Done in r95597.
- Doug
> On Jan 21, 2010, at 12:45 PM, Douglas Gregor wrote:
>
>>
>> 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
