This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
patch to fix PR72778
- From: Vladimir Makarov <vmakarov at redhat dot com>
- To: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 2 Aug 2016 16:59:33 -0400
- Subject: patch to fix PR72778
- Authentication-results: sourceware.org; auth=none
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72778
The patch was bootstrapped on x86-64.
Committed as rev. 23900
Index: ChangeLog
===================================================================
--- ChangeLog (revision 238999)
+++ ChangeLog (working copy)
@@ -1,5 +1,11 @@
2016-08-02 Vladimir Makarov <vmakarov@redhat.com>
+ PR middle-end/72778
+ * lra-spills.c (regno_in_use_p): New.
+ (lra_final_code_change): Use it.
+
+2016-08-02 Vladimir Makarov <vmakarov@redhat.com>
+
PR rtl-optimization/69847
* lra-int.h (struct lra-reg): Use restore_rtx instead of
restore_regno.
Index: lra-spills.c
===================================================================
--- lra-spills.c (revision 238991)
+++ lra-spills.c (working copy)
@@ -686,6 +686,20 @@ return_regno_p (unsigned int regno)
return false;
}
+/* Return true if REGNO is one of subsequent USE after INSN. */
+static bool
+regno_in_use_p (rtx_insn *insn, unsigned int regno)
+{
+ while ((insn = next_nondebug_insn (insn)) != NULL_RTX
+ && INSN_P (insn) && GET_CODE (PATTERN (insn)) == USE)
+ {
+ if (REG_P (XEXP (PATTERN (insn), 0))
+ && regno == REGNO (XEXP (PATTERN (insn), 0)))
+ return TRUE;
+ }
+ return false;
+}
+
/* Final change of pseudos got hard registers into the corresponding
hard registers and removing temporary clobbers. */
void
@@ -693,7 +707,7 @@ lra_final_code_change (void)
{
int i, hard_regno;
basic_block bb;
- rtx_insn *insn, *curr, *next_insn;
+ rtx_insn *insn, *curr;
int max_regno = max_reg_num ();
for (i = FIRST_PSEUDO_REGISTER; i < max_regno; i++)
@@ -728,11 +742,7 @@ lra_final_code_change (void)
&& REG_P (SET_SRC (pat)) && REG_P (SET_DEST (pat))
&& REGNO (SET_SRC (pat)) == REGNO (SET_DEST (pat))
&& (! return_regno_p (REGNO (SET_SRC (pat)))
- || (next_insn = next_nondebug_insn (insn)) == NULL_RTX
- || ! INSN_P (next_insn)
- || GET_CODE (PATTERN (next_insn)) != USE
- || ! REG_P (XEXP (PATTERN (next_insn), 0))
- || REGNO (SET_SRC (pat)) != REGNO (XEXP (PATTERN (next_insn), 0))))
+ || ! regno_in_use_p (insn, REGNO (SET_SRC (pat)))))
{
lra_invalidate_insn_data (insn);
delete_insn (insn);