On 05/14/2014 12:49 PM, Richard Sandiford wrote:
Jeff Law <l...@redhat.com> writes:
On 05/13/14 14:11, Sandra Loosemore wrote:
2014-05-13 Catherine Moore <c...@codesourcery.com>
Sandra Loosemore <san...@codesourcery.com>
gcc/
* config/mips/mips.c (mips_order_regs_for_local_alloc): Delete.
* config/mips/mips.h (ADJUST_REG_ALLOC_ORDER): Delete.
* config/mips/mips-protos.h (mips_order_regs_for_local_alloc): Delete.
OK for the trunk.
Would it be OK to hold off until after the switch to LRA? That patch
has been written and the MIPS parts approved, but we're waiting for
some legal things to be sorted out and for a fixed version of the LRA
EXTRA_MEMORY_CONSTRAINT patch. I just think it'd be better to tune this
sort of thing once that's done, rather than tune it against reload.
Richard, is it OK to commit this patch now that LRA is in, or do you
want to experiment some more with tuning first? I think we're all in
agreement that this is broken old code that should be removed regardless
of whether we do other things to tune REG_ALLOC_ORDER.
-Sandra
