This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: PR middle-end/40147: Memory leak in revision 147436
On Thu, May 14, 2009 at 8:45 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, May 14, 2009 at 05:18:51PM +0200, Jakub Jelinek wrote:
>> On Thu, May 14, 2009 at 05:14:00PM +0200, Jan Hubicka wrote:
>> > > We have a memory leark since memory_identifier_string isn't marked with
>> > > GTY(()). OK for trunk and 4.3/4.4 branches?
>> >
>> > OK, thanks
>>
>> Do you need GTY(()) markers in both places? ?AFAIK you need it either
>> in the header on the extern decl and in that case not on the definition
>> in the .c file, or you don't put an extern decl in a header, add GTY(())
>> marker to the local definition and ensure the .c file is scanner for GC (add
>> gt-* include at the end of the file, add it to the makefile list).
>>
There may be more memory leaks in ggc:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40151
Could someone verify them?
Thanks.
--
H.J.