[llvm-commits] [llvm] r171896 - in /llvm/trunk: include/llvm/Support/YAMLTraits.h lib/Support/YAMLTraits.cpp

[Chandler Carruth]

On Tue, Jan 8, 2013 at 1:04 PM, Nick Kledzik <kledzik at apple.com> wrote:

> Author: kledzik
> Date: Tue Jan  8 15:04:44 2013
> New Revision: 171896
>
> URL: http://llvm.org/viewvc/llvm-project?rev=171896&view=rev
> Log:
> Fix memory leak in YAML I/O.
>

Cool, thanks for looking at this.


> Stop using BumpPtrAllocator for HNodes because
> they have fields (vector, map) which require HNode
> destructors to be run.
>

Note that you could instead have all the fields also go on the
BumpPtrAllocator if this code is performance sensitive.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130108/afcfa2b7/attachment.html>