This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]