This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR middle-end/61141
- From: John David Anglin <dave dot anglin at bell dot net>
- To: Jeff Law <law at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 8 Jun 2014 14:06:40 -0400
- Subject: Re: [PATCH] Fix PR middle-end/61141
- Authentication-results: sourceware.org; auth=none
- References: <BLU0-SMTP43F10F32286A327F4C627297330 at phx dot gbl> <537A44A8 dot 3020502 at redhat dot com> <BLU0-SMTP827B721F0524FA46DDAB5F97320 at phx dot gbl> <537BC7D2 dot 1010705 at redhat dot com> <BLU0-SMTP66AB24AF0CF74AC7F5AAA897390 at phx dot gbl> <538ECCEC dot 2090808 at redhat dot com>
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 good to know. I scanned final and final_scan_insn and
see a couple places where we delete insns, but if I'm reading the
code correctly, they would result in seeing the NOTE_INSN_DELETED
rather than a real insn in the output routines.
So what code during final is deleting these insns?
It's the PA backend. For example,
/* If this isn't a sibcall, we put the load of %r27 into the
delay slot. We can't do this in a sibcall as we don't
have a second call-clobbered scratch register available.
*/
if (seq_length != 0
&& ! JUMP_P (NEXT_INSN (insn))
&& !sibcall)
{
final_scan_insn (NEXT_INSN (insn), asm_out_file,
optimize, 0, NULL);
/* Now delete the delay insn. */
SET_INSN_DELETED (NEXT_INSN (insn));
delay_insn_deleted = 1;
}
The insn is deleted so that it won't be emitted again. This is done
in a bunch
of places.
What's new is the verification after assembly output.
Dave
--
John David Anglin dave.anglin@bell.net