This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to fix PR55775
- From: David Miller <davem at davemloft dot net>
- To: vmakarov at redhat dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Fri, 21 Dec 2012 13:29:37 -0800 (PST)
- Subject: Re: patch to fix PR55775
- References: <50D4D307.109@redhat.com>
From: Vladimir Makarov <vmakarov@redhat.com>
Date: Fri, 21 Dec 2012 16:22:15 -0500
> 2012-12-21 Vladimir Makarov <vmakarov@redhat.com>
>
> PR middle-end/55775
> * lra-assigns.c (improve_inheritance): Do nothing after
> LRA_MAX_INHERITANCE_PASSES pass.
> * lra-constraints.c (MAX_CONSTRAINT_ITERATION_NUMBER): Rename to
> LRA_MAX_CONSTRAINT_ITERATION_NUMBER. Move to lra-int.h.
> (MAX_INHERITANCE_PASSES): Rename to LRA_MAX_INHERITANCE_PASSES.
> Move to lra-int.h.
> * lra-int.h (LRA_MAX_CONSTRAINT_ITERATION_NUMBER): Move from
> lra-constraints.c.
> (LRA_MAX_INHERITANCE_PASSES): Ditto.
The changes to lra_inheritabnce() and lra_undo_inheritance() are not
listed.
Also, why is it OK to simply ignore the fact that a relaxation
algorithm is still making changes? Shouldn't we be instead fixing
whatever prevents proper convergence?