Re: patch to fix PR 48435

[Graham Stott <graham.stott@btinternet.com>]

Vladimir,

The wrong PR 4435 was in referrenced the commit 

--- On Fri, 8/4/11, Jeff Law <law@redhat.com> wrote:

> From: Jeff Law <law@redhat.com>
> Subject: Re: patch to fix PR 48435
> To: "Vladimir Makarov" <vmakarov@redhat.com>
> Cc: "gcc-patches" <gcc-patches@gcc.gnu.org>
> Date: Friday, 8 April, 2011, 16:18
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 04/07/11 13:41, Vladimir Makarov wrote:
> > The following patch should solve problem
> > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48435. 
> It might solve other
> > latest IRA problems too including performance related
> ones.  The patch
> > is for targets which require some values to be placed
> in hard registers
> > starting with an even (odd) hard registers.
> > 
> > I already addressed an analogous problem recently and
> the patch I sent
> > that time although solved some problems it created
> even more new
> > problems.  I should acknowledge this.
> > 
> > The problem was in that profitable hard regs were used
> for colorability
> > criterion and finding hard registers where allocno
> values can be resided
> > but *also* as starting allocno hard registers. 
> It resulted in spilling
> > allocnos which should be placed in multi-registers
> starting on a
> > specific border because profitable hard registers were
> only starting
> > registers and when we calculated number of available
> hard registers
> > multi-register allocnos can not fit only in their
> starting hard registers.
> > 
> > The following patch was successfully bootstrapped on
> x86/x86-64 and on
> > i686 with H.J.'s autotester options.
> > 
> > OK to commit?
> > 
> > 2011-04-07  Vladimir Makarov <vmakarov@redhat.com>
> > 
> >     PR 4435
> >     * ira-color.c
> (setup_profitable_hard_regs): Add comments.
> >     Don't take prohibited hard
> regs into account.
> > 
>    (setup_conflict_profitable_regs): Rename
> to
> >     get_conflict_profitable_regs.
> >     (check_hard_reg_p): Check
> prohibited hard regs.
> OK.
> jeff
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
> 
> iQEcBAEBAgAGBQJNnycrAAoJEBRtltQi2kC70DsH/A7fnYlawxDvYPeULGb1ctQD
> 6LYWBWhJY4UDTiUNjLEzwALpq2XccVzubz1KEMI3wOndIHHYM2ykbsgTqCKmXqhf
> ZthBjzNUtE/i7jSxK8JK3SEShqs2j7vaAvlkUKbdF7O61rhQap8MRnD2umHfrbet
> PFea7MedpaC96mazzoyUWlzwWCt2CQhOlMFGlC44bxB1FrKy0nUjeIM7kWqjqG5W
> Y9lV8RlC4O2cLST5qHpqtssLSO4omPlKsEUBEKW4E/87UCoFAxP0n8NOZ1EFgMsc
> 3SlhSv+Dpinfh79gjHXt5+CH33bXTecf8Jfyd89Rn6huzL9OhAXJy+MjMi6cBm0=
> =KSaH
> -----END PGP SIGNATURE-----
>