Thanks for the comments, I will be sending a new patch and fixed changelog.

On 02/08/12 11:27, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos <pa...@matos-sorge.com> wrote:
Forgot to mention: this is to fix PR 54154.

Updated changelog:

2012-08-02   Paulo Matos <paulo.ma...@csr.com>

     PR middle-end/54154

     * regcprop.c (copy_value): remove check for redundant moves.
     * regcprop.c (copy_value): add check for redundant moves,
     remove instructions if redundant.

That's in copyprop_hardreg_forward_1

+  gcc_assert(dr != sr);
+

space before ().

+  FOR_BB_INSNS_SAFE(bb, insn, next)
      {

Likewise.

+        unsigned int dr = REGNO(SET_DEST(set));
+        unsigned int sr = REGNO(SET_SRC(set));
+

Likewise.

Richard.







--
PMatos

Reply via email to