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

2015-06-10 Thread Richard Sandiford
Carrot Wei car...@google.com 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. */

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 richard.sandif...@arm.com wrote: Carrot Wei car...@google.com writes: Index: simplify-got.c === --- simplify-got.c (revision 224174) +++ simplify-got.c (working copy) @@ -169,7

[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

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 car...@google.com 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