Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2015-01-14 Thread Martin Liška
On 12/22/2014 03:50 PM, Martin Liška wrote: On 12/18/2014 08:40 PM, Jan Hubicka wrote: 2014-12-08 Martin Liska * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * symbol-summary.h: New file. * gengtype.c (open_base_files): Add symbol-summary.h.

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-12-22 Thread Martin Liška
On 12/18/2014 08:40 PM, Jan Hubicka wrote: 2014-12-08 Martin Liska * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * symbol-summary.h: New file. * gengtype.c (open_base_files): Add symbol-summary.h. * toplev.c (general_init)

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-12-18 Thread Jan Hubicka
> 2014-12-08 Martin Liska > > * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID > is filled up. > * symbol-summary.h: New file. > * gengtype.c (open_base_files): Add symbol-summary.h. > * toplev.c (general_init): Call constructor of symbol_table. > ---

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-12-08 Thread Martin Liška
On 11/14/2014 04:23 PM, Martin Liška wrote: On 11/14/2014 03:04 PM, Martin Liška wrote: On 11/13/2014 04:50 PM, Jan Hubicka wrote: gcc/ChangeLog: 2014-11-12 Martin Liska * Makefile.in: New object file is added. * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is f

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-14 Thread Martin Liška
On 11/14/2014 03:04 PM, Martin Liška wrote: On 11/13/2014 04:50 PM, Jan Hubicka wrote: gcc/ChangeLog: 2014-11-12 Martin Liska * Makefile.in: New object file is added. * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * cgraph_summary.c: New file.

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-14 Thread Martin Liška
On 11/13/2014 04:50 PM, Jan Hubicka wrote: gcc/ChangeLog: 2014-11-12 Martin Liska * Makefile.in: New object file is added. * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * cgraph_summary.c: New file. * cgraph_summary.h: Ne

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-14 Thread Richard Biener
On Thu, Nov 13, 2014 at 4:50 PM, Jan Hubicka wrote: >> gcc/ChangeLog: >> >> 2014-11-12 Martin Liska >> >> * Makefile.in: New object file is added. >> * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID >> is filled up. >> * cgraph_summary.c: New file. >>

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Trevor Saunders
On Thu, Nov 13, 2014 at 11:23:31PM +0100, Markus Trippelsdorf wrote: > On 2014.11.14 at 01:19 -0500, Trevor Saunders wrote: > > On Thu, Nov 13, 2014 at 03:48:34PM +0100, Markus Trippelsdorf wrote: > > > On 2014.11.13 at 15:11 +0100, mliska wrote: > > > > > > > + /* Destructor. */ > > > > + virt

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Markus Trippelsdorf
On 2014.11.14 at 01:19 -0500, Trevor Saunders wrote: > On Thu, Nov 13, 2014 at 03:48:34PM +0100, Markus Trippelsdorf wrote: > > On 2014.11.13 at 15:11 +0100, mliska wrote: > > > > > + /* Destructor. */ > > > + virtual ~cgraph_summary () > > > + { > > > +destroy (); > > > + } > > > > From

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Trevor Saunders
On Thu, Nov 13, 2014 at 03:48:34PM +0100, Markus Trippelsdorf wrote: > On 2014.11.13 at 15:11 +0100, mliska wrote: > > Just two remarks: > > > +template > > +class GTY((user)) cgraph_summary > > +{ > > +public: > > + /* Default construction takes SYMTAB as an argument. */ > > + cgraph_summar

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Jan Hubicka
> gcc/ChangeLog: > > 2014-11-12 Martin Liska > > * Makefile.in: New object file is added. > * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID > is filled up. > * cgraph_summary.c: New file. > * cgraph_summary.h: New file. Since I am trying to get rid

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Martin Liška
On 11/13/2014 03:48 PM, Markus Trippelsdorf wrote: On 2014.11.13 at 15:11 +0100, mliska wrote: Just two remarks: +template +class GTY((user)) cgraph_summary +{ +public: + /* Default construction takes SYMTAB as an argument. */ + cgraph_summary (symbol_table *symtab, bool ggc = false): m_g

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Markus Trippelsdorf
On 2014.11.13 at 15:11 +0100, mliska wrote: Just two remarks: > +template > +class GTY((user)) cgraph_summary > +{ > +public: > + /* Default construction takes SYMTAB as an argument. */ > + cgraph_summary (symbol_table *symtab, bool ggc = false): m_ggc (ggc), > +m_insertion_enabled (true