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 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).

	Jakub


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