Re: [Patch] Fix PR54814

[Georg-Johann Lay <avr at gjlay dot de>]

Jeff Law schrieb:
> On 01/25/2013 11:41 AM, Georg-Johann Lay wrote:
> > PR54814 causes spill fails because reload.c:find_valid_class_1 tests 
> > only one
> > hard register instead of all hard registers of regno:mode in rclass:
> >
> > http://gcc.gnu.org/PR54814
> >
> >
> > The patch was originally worked out by Bernd Schmidt and fixed a problem
> > introduced in
> >
> > http://gcc.gnu.org/r190252
> >
> > The patch is bootstrapped and tested on x86_64-linux and also fixes 
> > the spill
> > fails that originally occurred on avr-unknown-one.
> >
> > Ok to apply?
> >
> >
> >     PR other/54814
> >     * reload.c (find_valid_class_1): Use in_hard_reg_set_p instead of
> >     TEST_HARD_REG_BIT.
> Is this a regression relative to a prior version of GCC?

Yes, it's 4.8 regression.  4.7 works and r190252 was only applied to 4.8 
trunk.

> If not, it'll probably need release manager approval before it can go in.
>
> Please attach your patch to PR54814 and attach PR 54814 to the 4.9 
> pending patches meta bug.

The patch *is* already attached, I can attach it again if that helps.

Johann