[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