Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-03-22 Thread Paolo Bonzini
Il 18/03/2013 01:01, Steven Bosscher ha scritto: >> > someone else DF aware anyway. If it is 4.9 material perhaps >> > it is simpler to place it after the patches that kill the basic >> > block argument. > Here is the updated patch. Bootstrapped&tested on > powerpc64-unknown-linux-gnu. OK for trun

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-03-22 Thread Steven Bosscher
On Mon, Mar 18, 2013 at 1:01 AM, Steven Bosscher wrote: > On Thu, Feb 21, 2013 at 1:33 PM, Paolo Bonzini wrote: >> someone else DF aware anyway. If it is 4.9 material perhaps >> it is simpler to place it after the patches that kill the basic >> block argument. > > Here is the updated patch. Bootst

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-02-21 Thread Paolo Bonzini
> > so it needs someone DF aware to review > > and that makes it stage1 material as well I think. > > Also good. I think the patch is quite safe but it's obviously not a > bug fix (except perhaps for making the dumps less confusing). Let's > consider this queued for GCC 4.9. > FWIW, I like to thi

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-02-21 Thread Steven Bosscher
On Thu, Feb 21, 2013 at 10:59 AM, Richard Biener wrote: > On Thu, Feb 21, 2013 at 1:10 AM, Steven Bosscher wrote: > +/* Delete all of the refs information from the insn with UID. > + Internal helper for df_insn_info_delete, df_insn_rescan, and other > > df_insn_delete I suppose Right. > + df

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-02-21 Thread Richard Biener
On Thu, Feb 21, 2013 at 1:10 AM, Steven Bosscher wrote: > Hello, > > The attached patch splits a new function df_insn_info_delete from > df_insn_delete. The original motivation was to get rid of the silly > "deleting insn with uid = ..." messages when re-scanning an insn, > because the mentioned i