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

Török Edwin edwintorok at gmail.com
Sun May 4 01:52:27 PDT 2008


Chris Lattner wrote:
> 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!

Thanks, it works now!

--Edwin





More information about the llvm-commits mailing list