Re: [PATCH 0/2] Port symtab/cgraph/varpool nodes to use C++ inheritance

2013-08-26 Thread David Malcolm
On Tue, 2013-08-20 at 23:01 +0200, Jan Hubicka wrote: [...] > > > > In summary, > > struct GTY(()) symtab_node_base > > becomes: > > class GTY((user)) symtab_node_base > > > > and the subclasses: > > struct GTY(()) cgraph_node > > and: > > struct GTY(()) varpool_node > > > > become (resp

Re: [PATCH 0/2] Port symtab/cgraph/varpool nodes to use C++ inheritance

2013-08-21 Thread Martin Jambor
Hi, On Tue, Aug 20, 2013 at 11:01:04PM +0200, Jan Hubicka wrote: [...] > > Currently to access the base symtab fields of a cgraph or varpool > > node, the code has e.g. > > > >node->symbol.decl > > > > whereas with C++ inheritance, the "symbol" field is no more, and we > >

Re: [PATCH 0/2] Port symtab/cgraph/varpool nodes to use C++ inheritance

2013-08-20 Thread Jan Hubicka
Hi, sorry for late reply, I noticed the patch only yesterday. > Honza: following up from our IRC chat, I've ported the symtab, cgraph > and varpool nodes from the current hand-coded inheritance-in-C scheme > to being a C+ class hierarchy. > > I know you're in the middle of making lots of other ch

[PATCH 0/2] Port symtab/cgraph/varpool nodes to use C++ inheritance

2013-08-15 Thread David Malcolm
Honza: following up from our IRC chat, I've ported the symtab, cgraph and varpool nodes from the current hand-coded inheritance-in-C scheme to being a C+ class hierarchy. I know you're in the middle of making lots of other changes to this code, so I've written a script to automate the parts of the