Re: RFA: another patch to solve PR49154

2011-06-02 Thread Hans-Peter Nilsson
On Tue, 31 May 2011, Richard Sandiford wrote: > Gah, seems like I'd forgotten the "no subclasses" bit by the time > I started looking at code. Sorry for the false alarm. Still, the extra look made me realise that I should have restricted that statement to allocatable registers. (And I really do

Re: RFA: another patch to solve PR49154

2011-05-31 Thread Richard Sandiford
Hans-Peter Nilsson writes: > On Tue, 31 May 2011, Richard Sandiford wrote: >> Hans-Peter Nilsson writes: >> > Index: tm.texi.in >> > === >> > --- tm.texi.in (revision 174376) >> > +++ tm.texi.in (working copy) >> > @@ -2327,6

Re: RFA: another patch to solve PR49154

2011-05-31 Thread Hans-Peter Nilsson
On Tue, 31 May 2011, Richard Sandiford wrote: > Hans-Peter Nilsson writes: > > Index: tm.texi.in > > === > > --- tm.texi.in (revision 174376) > > +++ tm.texi.in (working copy) > > @@ -2327,6 +2327,11 @@ constraints is throu

Re: RFA: another patch to solve PR49154

2011-05-31 Thread Richard Sandiford
Hans-Peter Nilsson writes: > Index: tm.texi.in > === > --- tm.texi.in(revision 174376) > +++ tm.texi.in(working copy) > @@ -2327,6 +2327,11 @@ constraints is through machine-dependent > You can define such letters to

Re: RFA: another patch to solve PR49154

2011-05-27 Thread Hans-Peter Nilsson
On Sat, 28 May 2011, Hans-Peter Nilsson wrote: > +You must define the narrowest register class for a register so that > +class either has no subclasses, or that for some mode, the move cost > +between registers within the class are cheaper than moving a register Bah: "is cheaper" is the grammatica

Re: RFA: another patch to solve PR49154

2011-05-27 Thread Hans-Peter Nilsson
On Thu, 26 May 2011, Vladimir Makarov wrote: > On 05/26/2011 04:47 AM, Hans-Peter Nilsson wrote: > > On Wed, 25 May 2011, Vladimir Makarov wrote: > > It sounds like you're saying that "the narrowest register > > classes must be formed of registers for which there are trivial > > instructions to mov

Re: RFA: another patch to solve PR49154

2011-05-27 Thread Hans-Peter Nilsson
On Thu, 26 May 2011, Hans-Peter Nilsson wrote: > Good, a way forward (well, for me) even though the cure is > suspiciously incidental. Thanks for your patience. JFTR, setting CRIS_CC0_REGNUM to fixed just moves the assert to trig at the same place in the v10 multilib, where CRIS_MOF_REGNUM is ava

Re: RFA: another patch to solve PR49154

2011-05-26 Thread Hans-Peter Nilsson
On Thu, 26 May 2011, Vladimir Makarov wrote: > On 05/26/2011 04:47 AM, Hans-Peter Nilsson wrote: > > On Wed, 25 May 2011, Vladimir Makarov wrote: > > It sounds like you're saying that "the narrowest register > > classes must be formed of registers for which there are trivial > > instructions to mov

Re: RFA: another patch to solve PR49154

2011-05-26 Thread Vladimir Makarov
On 05/26/2011 04:47 AM, Hans-Peter Nilsson wrote: On Wed, 25 May 2011, Vladimir Makarov wrote: On 11-05-25 6:58 PM, Hans-Peter Nilsson wrote: On Wed, 25 May 2011, Vladimir Makarov wrote: This patch solves http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49154 for CRIS. The problem was in that the

Re: RFA: another patch to solve PR49154

2011-05-26 Thread Hans-Peter Nilsson
On Wed, 25 May 2011, Vladimir Makarov wrote: > On 11-05-25 6:58 PM, Hans-Peter Nilsson wrote: > > On Wed, 25 May 2011, Vladimir Makarov wrote: > > > > > This patch solves http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49154 for > > > CRIS. > > > The problem was in that the pressure classes did not con

Re: RFA: another patch to solve PR49154

2011-05-25 Thread Vladimir Makarov
On 11-05-25 6:58 PM, Hans-Peter Nilsson wrote: On Wed, 25 May 2011, Vladimir Makarov wrote: This patch solves http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49154 for CRIS. The problem was in that the pressure classes did not contain SRP register and the assert failed. I'm not sure I understand t

Re: RFA: another patch to solve PR49154

2011-05-25 Thread Hans-Peter Nilsson
On Wed, 25 May 2011, Vladimir Makarov wrote: > This patch solves http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49154 for CRIS. > The problem was in that the pressure classes did not contain SRP register and > the assert failed. I'm not sure I understand the basic requirement. > OK to commit? > > 2

RFA: another patch to solve PR49154

2011-05-25 Thread Vladimir Makarov
This patch solves http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49154 for CRIS. The problem was in that the pressure classes did not contain SRP register and the assert failed. OK to commit? 2011-05-25 Vladimir Makarov PR rtl-optimization/49154 * config/cris/cris.h (reg_class