Re: [patch] PR66714 -- Re: Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-24 Thread Jakub Jelinek
On Fri, Jul 24, 2015 at 07:21:07AM -0700, Cesar Philippidis wrote: > This patch the check for IS_TYPE_OF_DECL_P in this patch. Is this ok for > trunk? > 2015-07-24 Cesar Philippidis > > gcc/ > * tree-cfg.c (struct replace_decls_d): New struct. > (replace_block_vars_by_duplica

Re: [patch] PR66714 -- Re: Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-24 Thread Cesar Philippidis
On 07/23/2015 03:11 PM, Jakub Jelinek wrote: > On Thu, Jul 23, 2015 at 03:01:25PM -0700, Cesar Philippidis wrote: >> On 07/23/2015 08:32 AM, Jakub Jelinek wrote: >>> On Thu, Jul 23, 2015 at 08:20:50AM -0700, Cesar Philippidis wrote: The attached patch does just that; it teaches replace_bl

Re: [patch] PR66714 -- Re: Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-23 Thread Jakub Jelinek
On Thu, Jul 23, 2015 at 03:01:25PM -0700, Cesar Philippidis wrote: > On 07/23/2015 08:32 AM, Jakub Jelinek wrote: > > On Thu, Jul 23, 2015 at 08:20:50AM -0700, Cesar Philippidis wrote: > >> The attached patch does just that; it teaches > >> replace_block_vars_by_duplicates to replace the decls insi

Re: [patch] PR66714 -- Re: Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-23 Thread Cesar Philippidis
On 07/23/2015 08:32 AM, Jakub Jelinek wrote: > On Thu, Jul 23, 2015 at 08:20:50AM -0700, Cesar Philippidis wrote: >> The attached patch does just that; it teaches >> replace_block_vars_by_duplicates to replace the decls inside the >> value-exprs with a duplicate too. It's kind of messy though. At t

Re: [patch] PR66714 -- Re: Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-23 Thread Jakub Jelinek
On Thu, Jul 23, 2015 at 08:20:50AM -0700, Cesar Philippidis wrote: > The attached patch does just that; it teaches > replace_block_vars_by_duplicates to replace the decls inside the > value-exprs with a duplicate too. It's kind of messy though. At the > moment I'm only considering VAR_DECL, PARM_DE