Re: [PATCH] Fix PR middle-end/61141

2014-06-09 Thread John David Anglin
On 6/9/2014 12:41 PM, Jeff Law wrote: What's new is the verification after assembly output. Right. Thanks for the analysis. I know it takes significant time. Not a problem. Thanks for installing the change. Dave -- John David Anglindave.ang...@bell.net

Re: [PATCH] Fix PR middle-end/61141

2014-06-09 Thread Jeff Law
On 06/08/14 12:06, John David Anglin wrote: On 4-Jun-14, at 3:38 AM, Jeff Law wrote: On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence con

Re: [PATCH] Fix PR middle-end/61141

2014-06-08 Thread Jeff Law
On 06/08/14 12:06, John David Anglin wrote: On 4-Jun-14, at 3:38 AM, Jeff Law wrote: On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence con

Re: [PATCH] Fix PR middle-end/61141

2014-06-08 Thread John David Anglin
On 4-Jun-14, at 3:38 AM, Jeff Law wrote: On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence containing a NOTE_INSN_DELETED. OK. That's go

Re: [PATCH] Fix PR middle-end/61141

2014-06-04 Thread Jeff Law
On 05/24/14 12:39, John David Anglin wrote: The insns are being deleted in "final" after the assembly output has being done. So, for example, pa_output_call is never called with a delayed branch sequence containing a NOTE_INSN_DELETED. OK. That's good to know. I scanned final and final_scan_i

Re: [PATCH] Fix PR middle-end/61141

2014-05-24 Thread John David Anglin
On 20-May-14, at 5:23 PM, Jeff Law wrote: I believe that the backend must handle the deleted insns in the delay slot as there are are no compilation errors or regressions. However, like you, I'm not 100% certain this done correctly. I'm pretty sure we're getting this wrong in the backend. In f

Re: [PATCH] Fix PR middle-end/61141

2014-05-20 Thread Jeff Law
On 05/19/14 15:20, John David Anglin wrote: The problem compiling c-common.c is very hard to debug. These routines are called intensively and the ICE occurs after several million calls. It takes a couple of hours of running under gdb to reach the failing call to reset_insn_used_flags just counti

Re: [PATCH] Fix PR middle-end/61141

2014-05-19 Thread John David Anglin
Hi Jeff, On 19-May-14, at 1:51 PM, Jeff Law wrote: On 05/18/14 09:33, John David Anglin wrote: The attached change appears to fix PR middle-end/61141. On PA, we can get deleted insn notes in call sequences. The attached change checks to make sure we have a valid insn before calling reset_i

Re: [PATCH] Fix PR middle-end/61141

2014-05-19 Thread Jeff Law
On 05/18/14 09:33, John David Anglin wrote: The attached change appears to fix PR middle-end/61141. On PA, we can get deleted insn notes in call sequences. The attached change checks to make sure we have a valid insn before calling reset_insn_used_flags and verify_insn_sharing. Tested on hppa-

[PATCH] Fix PR middle-end/61141

2014-05-18 Thread John David Anglin
The attached change appears to fix PR middle-end/61141. On PA, we can get deleted insn notes in call sequences. The attached change checks to make sure we have a valid insn before calling reset_insn_used_flags and verify_insn_sharing. Tested on hppa-unknown-linux-gnu, hppa2.0w-hp-hpux11.1