This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: another patch to solve PR49154
- From: Hans-Peter Nilsson <hp at bitrange dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 25 May 2011 18:58:42 -0400 (EDT)
- Subject: Re: RFA: another patch to solve PR49154
- References: <4DDD7705.9020003@redhat.com>
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?
>
> 2011-05-25 Vladimir Makarov <vmakarov@redhat.com>
>
> PR rtl-optimization/49154
> * config/cris/cris.h (reg_class, REG_CLASS_NAMES)
> (REG_CLASS_CONTENTS, REGNO_REG_CLASS): Add class SRP_REGS.
Can you please suggest an update to the target macro
documentation to reflect the requirement it's currently failing?
To feel ok with this change I need to understand why it's not ok
as is: I can't see the error, just a random change narrowing a
register class that at a glance *should* not be needed. To wit,
I need to understand why the bug is here and that it's not the
failing assert in IRA that's wrong.
(And thanks for your work, really. Don't worry about fitting
the update into texinfo format, I can help with that.)
brgds, H-P