Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-23 Thread Eric Botcazou
> It was also on the mailing list too. If I'd had the reference to 57940, > I wouldn't have approved the patch given your comment from July 20. Understood. I missed the message on the list and remembered only the PR. -- Eric Botcazou

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread Jeff Law
On 08/21/2013 11:32 AM, Eric Botcazou wrote: (I can't find the original mail either in my mailbox or in the archives). It's PR rtl-optimization/57940. It was also on the mailing list too. If I'd had the reference to 57940, I wouldn't have approved the patch given your comment from July 20.

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread Eric Botcazou
> (I can't find the original mail either in my mailbox or in the archives). It's PR rtl-optimization/57940. -- Eric Botcazou

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread Steven Bosscher
On Wed, Aug 21, 2013 at 5:10 PM, Jeff Law wrote: > On 08/21/2013 08:25 AM, David Edelsohn wrote: >> >> This patch has caused a bootstrap failure for powerpc-aix and probably >> powerpc64-linux. GCC segfaults and core dumps during stage2 >> configure. >> >> The motivation for this patch seems faul

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread Jeff Law
On 08/21/2013 08:25 AM, David Edelsohn wrote: This patch has caused a bootstrap failure for powerpc-aix and probably powerpc64-linux. GCC segfaults and core dumps during stage2 configure. The motivation for this patch seems faulty and I strongly request that it be reverted. Instead of going ins

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread David Edelsohn
This patch has caused a bootstrap failure for powerpc-aix and probably powerpc64-linux. GCC segfaults and core dumps during stage2 configure. The motivation for this patch seems faulty and I strongly request that it be reverted. PR bootstrap/58206 Thanks, David

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-21 Thread Bernd Schmidt
On 08/19/2013 11:05 PM, Jeff Law wrote: > On 07/20/2013 03:02 AM, Alexey Makhalov wrote: >> Hello! >> >> If delete_unmarked_insns deletes some insn, DF state might be >> out of date, and, regs_ever_live might contain unused registers till >> the end. (I can't find the original mail either in my ma

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-20 Thread Jeff Law
On 07/20/2013 03:02 AM, Alexey Makhalov wrote: 2013-07-20 Alexey Makhalov mailto:makhal...@gmail.com>> * dce.c (fini_dce): Call df_analyze again just in case delete_unmarked_insns removed anything. Thanks. Installed. jeff

Re: [PATCH] Rerun df_analyze after delete_unmarked_insns during DCE

2013-08-19 Thread Jeff Law
On 07/20/2013 03:02 AM, Alexey Makhalov wrote: Hello! If delete_unmarked_insns deletes some insn, DF state might be out of date, and, regs_ever_live might contain unused registers till the end. Fixed by forcing regs_ever_live update and rerunning df_analyze () at fini_dce(). I found this bug