[PATCH] Ensure entries pid_map are initialized to NULL

Diego Novillo dnovillo@google.com
Wed Aug 5 21:54:00 GMT 2009


On Wed, Aug 5, 2009 at 17:52, Xinliang David Li<davidxl@google.com> wrote:

>> I've just noticed that we never free pid_map.  What would be a good
>> place to do that?  This is not related to this change, so if you find
>> a good spot, it can be done in a separate patch.
>
> This is not uncommon for objects with global life time. Other examples
> include cgraph nodes, varpool nodes etc.

No.  These live in GC memory.  pid_map lives on the heap.  We leak
memory left-right-and-center, so I don't think it's critical.


Diego.



More information about the Gcc-patches mailing list