Re: [PATCH, Google] Notify df framework when removing an insn in simplify-got

2015-06-10 Thread Carrot Wei
On Tue, Jun 9, 2015 at 11:43 PM, Richard Sandiford wrote: > Carrot Wei writes: >> Index: simplify-got.c >> === >> --- simplify-got.c (revision 224174) >> +++ simplify-got.c (working copy) >> @@ -169,7 +169,10 @@ >> >>/* Since the

Re: [PATCH, Google] Notify df framework when removing an insn in simplify-got

2015-06-09 Thread Richard Sandiford
Carrot Wei writes: > Index: simplify-got.c > === > --- simplify-got.c (revision 224174) > +++ simplify-got.c (working copy) > @@ -169,7 +169,10 @@ > >/* Since there is no usage of pic_reg now, we can remove it. */ >if (use) >

Re: [PATCH, Google] Notify df framework when removing an insn in simplify-got

2015-06-09 Thread Xinliang David Li
ok. David On Tue, Jun 9, 2015 at 4:46 PM, Carrot Wei wrote: > Hi > > I forgot to notify df framework when I removed an insn, it caused df > verification failure described in google bug b/16155462. > > The following patch passed regression test on arm qemu in both thumb > and arm modes. > OK for

[PATCH, Google] Notify df framework when removing an insn in simplify-got

2015-06-09 Thread Carrot Wei
Hi I forgot to notify df framework when I removed an insn, it caused df verification failure described in google bug b/16155462. The following patch passed regression test on arm qemu in both thumb and arm modes. OK for google 4.9 branch? Index: simplify-got.c ==