Jiong Wang writes:

> Andreas Schwab writes:
>
>> Jiong Wang <jiong.w...@arm.com> writes:
>>
>>>   I Just finished several round of rebuild & testing on clean
>>>   environment.
>>
>> How did you even manage to compile it?
>>
>> ../../gcc/ira.c: In function 'void print_translated_classes(FILE*, bool)':
>> ../../gcc/ira.c:1415:49: error: iteration 8u invokes undefined behavior 
>> [-Werror=aggressive-loop-optimizations]
>>      fprintf (f, " %s -> %s\n", reg_class_names[i],
>>                                                  ^
>> ../../gcc/ira.c:1414:17: note: containing loop
>>    for (i = 0; i < N_REG_CLASSES; i++)
>
> And, my new patch missed a ',' after "FIXED_REG0", sorry about
> that.. that should be the problem for the bootstrap issue.

committed patch below as obivious after bootstrap OK and check-go OK.

2015-08-11  Jiong Wang  <jiong.w...@arm.com>

gcc/
  * config/aarch64/aarch64.h (REG_CLASS_NAMES): Add the missing ',' after
  FIXED_REG0.

diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
index 43ff895..d3ae393 100644
--- a/gcc/config/aarch64/aarch64.h
+++ b/gcc/config/aarch64/aarch64.h
@@ -417,7 +417,7 @@ enum reg_class
 #define REG_CLASS_NAMES				\
 {						\
   "NO_REGS",					\
-  "FIXED_REG0"					\
+  "FIXED_REG0",					\
   "CALLER_SAVE_REGS",				\
   "GENERAL_REGS",				\
   "STACK_REG",					\

Reply via email to