Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-16 Thread nick clifton
Hi Mike, Hi Steve, * config/rx/rx.h: Remove SMALL_REGISTER_CLASSES. I wasn't sure if these are obvious, because I already removed the definition from rx.h before, see ChangeLog-2010. Maybe someone put it back intentionally. Nick did it (intentionally) with: Sorry about that - sanfu

Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-15 Thread Mike Stump
On Feb 15, 2012, at 11:09 AM, Steven Bosscher wrote: > On Wed, Feb 15, 2012 at 7:59 PM, DJ Delorie wrote: >> >> I would classify these as "obvious" but please make sure they go in >> before the poison patch ;-) >> >>> And this will be needed to make those ports do what was intended: >>> >>>

Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-15 Thread Steven Bosscher
On Wed, Feb 15, 2012 at 7:59 PM, DJ Delorie wrote: > > I would classify these as "obvious" but please make sure they go in > before the poison patch ;-) > >> And this will be needed to make those ports do what was intended: >> >>       * config/rl78/rl78.h: Replace SMALL_REGISTER_CLASSES with hook

Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-15 Thread DJ Delorie
I would classify these as "obvious" but please make sure they go in before the poison patch ;-) > And this will be needed to make those ports do what was intended: > > * config/rl78/rl78.h: Replace SMALL_REGISTER_CLASSES with hook. > * config/rx/rx.h: Remove SMALL_REGISTER_CLASSES. >

Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-15 Thread Steven Bosscher
On Wed, Feb 15, 2012 at 7:36 PM, Steven Bosscher wrote: > Hello, > > While going through some old patches of mine, I noticed this: > > config/rl78/rl78.h:#define SMALL_REGISTER_CLASSES               1 > config/rx/rx.h:#define SMALL_REGISTER_CLASSES           0 > > Can't possibly work, that target

[patch] Poison SMALL_REGISTER_CLASSES

2012-02-15 Thread Steven Bosscher
Hello, While going through some old patches of mine, I noticed this: config/rl78/rl78.h:#define SMALL_REGISTER_CLASSES 1 config/rx/rx.h:#define SMALL_REGISTER_CLASSES 0 Can't possibly work, that target macro was replaced with a hook. Attached patch OK for trunk? Ciao! S