This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Minor improvement to double-word register allocations


On 05/21/2010 11:31 AM, Jeff Law wrote:
On 05/21/10 09:29, Chung-Lin Tang wrote:
Hi there,
the ARM build of trunk also fails when compiling libgcc...

Judging from the comments of the ira_class_hard_reg_index array (negative as unavailable),
it seems that you probably meant this:


Index: ira-costs.c
===================================================================
--- ira-costs.c (revision 159662)
+++ ira-costs.c (working copy)
@@ -1794,7 +1794,7 @@
{
regno = ira_non_ordered_class_hard_regs[cover_class][j];
index = ira_class_hard_reg_index[cover_class][regno];
- ira_assert (index != 0);
+ ira_assert (index >= 0);
reg_costs[index] += ALLOCNO_FREQ (a);
}
}


I haven't tested it, but ARM on trunk now builds again...
Yes. Alternately index != -1
Jeff and Chung-Lin, the patch is ok. I think it was my typo, sorry.

It can be committed to the trunk.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]