[PATCH] D37750: [llvm-objdump] Fix memory leaks in macho dump
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 25 13:52:04 PDT 2017
Ping
On Mon, Sep 18, 2017 at 11:30 AM David Blaikie via Phabricator via
llvm-commits <llvm-commits at lists.llvm.org> wrote:
> dblaikie added inline comments.
>
>
> ================
> Comment at: llvm/trunk/tools/llvm-objdump/MachODump.cpp:5884
> outs() << "\n";
> + xar_iter_free(xp);
> }
> ----------------
> Any chance of an RAII API for this resource, to avoid the accident-prone
> manual cleanup/free?
>
>
> Repository:
> rL LLVM
>
> https://reviews.llvm.org/D37750
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170925/f90aa773/attachment.html>
More information about the llvm-commits
mailing list