[PATCH] D37750: [llvm-objdump] Fix memory leaks in macho dump

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 11:30:48 PDT 2017


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





More information about the llvm-commits mailing list