[llvm-commits] [PATCH] PostDominators: avoid memory leak

Chris Lattner clattner at apple.com
Sat May 3 16:45:41 PDT 2008


On May 3, 2008, at 1:35 PM, Török Edwin wrote:

> Bill Wendling wrote:
>> On May 3, 2008, at 8:56 AM, Török Edwin wrote:
>>
>>
>>> Hi,
>>>
>>> The DT  member of PostDominatorTree is never freed, and valgrind
>>> reports
>>> a leak.
>>> No other optimization passess leak, and I like to see as few  
>>> errors as
>>> possible from valgrind, hence this patch.
>>> It is only a matter of adding a simple destructor.
>>>
>>> Can I commit?
>>>
>>>
>> Looks good to me. Please commit.
>
> Thanks commited in r50607.
>
> BTW my commit messages never reach this list (I consulted the archives
> at lists.cs.uiuc.edu), and it is not just this commit.
> My previous commits don't show up either ...

Try another test commit please.  Thanks!

-Chris



More information about the llvm-commits mailing list