Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jan Hubicka wrote: > > On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > > > Hi, > > > here is updated patch. It has same effect as the former version. > > > > > > Bootstrapped/regtested x86_64-linux, OK? > > > > But what about the comment? > > > > We track no > >

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 01:42:00PM +0100, Jan Hubicka wrote: > > One solution I do envision, which might help, would be to try to figure > > out which types are unused, and discard those. Say, scan all variables > > within a lexical scope (including nested blocks), deciding which ones > > can be d

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
> > On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > > > Hi, > > > here is updated patch. It has same effect as the former version. > > > > > > Bootstrapped/regtested x86_64-linux, OK? > > > > But what about the comment? > > > > We track no > > information on whether given type i

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
> On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > Hi, > > here is updated patch. It has same effect as the former version. > > > > Bootstrapped/regtested x86_64-linux, OK? > > But what about the comment? > > We track no > information on whether given type is used or not, so we h

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jan Hubicka wrote: > Hi, > here is updated patch. It has same effect as the former version. > > Bootstrapped/regtested x86_64-linux, OK? But what about the comment? We track no information on whether given type is used or not, so we have to keep t

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
Hi, here is updated patch. It has same effect as the former version. Bootstrapped/regtested x86_64-linux, OK? Honza * tree-ssa-live.c (remove_unused_scope_block_p): Also remove reudndant typedefs. Index: tree-ssa-live.c

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jan Hubicka
> >Usually the things stay to make DECL_UID stable across -g and -g0. > > Only DECL_UID order needs to be stable I think. Well, when you remove a reference to a declaration, the DECL_UID order will change. We stream declarations in the order they are inserted to the encoder and that depends on

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jakub Jelinek
On Sat, Jan 16, 2016 at 11:31:49AM +0100, Richard Biener wrote: > On January 16, 2016 11:27:09 AM GMT+01:00, Jan Hubicka wrote: > >> On Fri, 15 Jan 2016, Jan Hubicka wrote: > >> > >> > Hi, > >> > this is last of my patches to improve partitionability of programs. > > > >> > This patch compacts BL

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Richard Biener
On January 16, 2016 11:27:09 AM GMT+01:00, Jan Hubicka wrote: >> On Fri, 15 Jan 2016, Jan Hubicka wrote: >> >> > Hi, >> > this is last of my patches to improve partitionability of programs. > >> > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for >-g0 >> > and redundant TYPE_DECL

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jan Hubicka
> On Fri, 15 Jan 2016, Jan Hubicka wrote: > > > Hi, > > this is last of my patches to improve partitionability of programs. > > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for -g0 > > and redundant TYPE_DECLs on all levels. It does make noticeable difference > > on firefox, but

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-15 Thread Richard Biener
On Fri, 15 Jan 2016, Jan Hubicka wrote: > Hi, > this is last of my patches to improve partitionability of programs. > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for -g0 > and redundant TYPE_DECLs on all levels. It does make noticeable difference > on firefox, but I managed to

Prune BLOCK_VARs lists in free_lang_data

2016-01-15 Thread Jan Hubicka
Hi, this is last of my patches to improve partitionability of programs. This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for -g0 and redundant TYPE_DECLs on all levels. It does make noticeable difference on firefox, but I managed to erase the numbers (can re-test them if requested)