[cfe-commits] r101194 - /cfe/trunk/include/clang/AST/ASTVector.h

[Ted Kremenek]

On Apr 14, 2010, at 10:57 AM, Daniel Dunbar wrote:

> On Wed, Apr 14, 2010 at 8:45 AM, Douglas Gregor <dgregor at apple.com> wrote:
>> 
>> On Apr 14, 2010, at 8:45 AM, Ted Kremenek wrote:
>> 
>>> I agree that a 'vector' shouldn't be used at all with InitListExpr.  My immediate interest was plugging the memory leak caused by using 'std::vector' without waiting for such a rewrite.  If we reach a point where vectors are not present in our AST elements, I think ASTVector should probably just get ripped out.
>> 
>> Sounds like a good plan.
> 
> Please file some kind of bug to actively track this cleanup.
> 
> - Daniel

Filed:

Bug 6838 - Change InitListExpr to not use a vector for subexpressions
Bug 6839 - Remove ASTVector once it is no longer needed 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20100414/1af9f191/attachment.html>