This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] rtl-optimization/71709, strcpy arg optimised out
- From: Alan Modra <amodra at gmail dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Fri, 1 Jul 2016 16:49:56 +0930
- Subject: [PATCH] rtl-optimization/71709, strcpy arg optimised out
- Authentication-results: sourceware.org; auth=none
This patch fixes a thinko in find_call_crossed_cheap_reg.
For functions that return an argument unchanged, like strcat,
find_call_crossed_cheap_reg attempts to find an assignment between
a pseudo reg and the arg reg before the call, so that uses of the
pseudo after the call can instead use the return value. The exit
condition on the loop looking at previous insns was wrong. Uses of
the arg reg don't matter. What matters is the insn setting the arg
reg as any assignment involving the arg reg prior to that insn is
likely a completely unrelated use of the hard reg.
Bootstrapped and regression tested powerpc64le-linux and x86_64-linux.
OK to apply?
PR rtl-optimization/71709
* ira-lives.c (find_call_crossed_cheap_reg): Exit loop on arg reg
being set, not referenced.
diff --git a/gcc/ira-lives.c b/gcc/ira-lives.c
index 6950ffb..6b7ee81 100644
--- a/gcc/ira-lives.c
+++ b/gcc/ira-lives.c
@@ -1014,7 +1014,7 @@ find_call_crossed_cheap_reg (rtx_insn *insn)
break;
}
- if (reg_overlap_mentioned_p (reg, PATTERN (prev)))
+ if (reg_set_p (reg, prev))
break;
}
prev = PREV_INSN (prev);
--
Alan Modra
Australia Development Lab, IBM