From patchwork Thu Oct 11 15:35:55 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [lra] patch to fix GCC crash on a SPEC2006 test
Date: Thu, 11 Oct 2012 05:35:55 -0000
From: Vladimir Makarov <vmakarov@redhat.com>
X-Patchwork-Id: 190931
Message-Id: <5076E75B.2000206@redhat.com>
To: gcc-patches <gcc-patches@gcc.gnu.org>

The following patch fixes scheduler crash on a SPEC2006 test because LRA 
did not process CALL_INSN_FUNCTION_USAGE.

The patch was successfully bootstrapped and tested on x86/x86-64.

Committed as rev. 192362.

2012-10-11  Vladimir Makarov  <vmakarov@redhat.com>

         * lra-spills.c (spill_pseudos): Process CALL_INSN_FUNCTION_USAGE
         too.


Index: lra-spills.c
===================================================================
--- lra-spills.c	(revision 192341)
+++ lra-spills.c	(working copy)
@@ -480,6 +480,8 @@ spill_pseudos (void)
 	if (bitmap_bit_p (&changed_insns, INSN_UID (insn)))
 	  {
 	    remove_pseudos (&PATTERN (insn), insn);
+	    if (CALL_P (insn))
+	      remove_pseudos (&CALL_INSN_FUNCTION_USAGE (insn), insn);
 	    if (lra_dump_file != NULL)
 	      fprintf (lra_dump_file,
 		       "Changing spilled pseudos to memory in insn #%u\n",
@@ -488,6 +490,12 @@ spill_pseudos (void)
 	    if (lra_reg_spill_p || targetm.different_addr_displacement_p ())
 	      lra_set_used_insn_alternative (insn, -1);
 	  }
+	else if (CALL_P (insn))
+	  /* Call insn might have not references for pseudos besides
+	     in CALL_INSN_FUNCTION_USAGE but we don't count them in
+	     insn_bitmap of corresponding lra_reg_info as they don't
+	     need reloads.  */
+	  remove_pseudos (&CALL_INSN_FUNCTION_USAGE (insn), insn);
       bitmap_and_compl_into (DF_LR_IN (bb), &spilled_pseudos);
       bitmap_and_compl_into (DF_LR_OUT (bb), &spilled_pseudos);
     }