This is the mail archive of the gcc@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: strange code in ra-build.c:handle_asm_insn


Joern Rennecke <joern.rennecke@superh.com> writes:

> Why do you call REG_CLASS_FROM_LETTER when you encounter digits
> between 5 and 9?
> 
>               case '=': case '+': case '*': case '%': case '?': case '!':
>               case '0': case '1': case '2': case '3': case '4': case 'm':
>               case '<': case '>': case 'V': case 'o': case '&': case 'E':
>               case 'F': case 's': case 'i': case 'n': case 'X': case 'I':
>               case 'J': case 'K': case 'L': case 'M': case 'N': case 'O':
>               case 'P':
>                 break;
> ...
>               default:
>                 cls =
>                   (int) reg_class_subunion[cls][(int)
>                                                 REG_CLASS_FROM_LETTER (c)];

IMHO: The handle_asm_insn used only for debugging.
IMHO: Michael just copied and modified  reload1.c:maybe_fix_stack_asms.
maybe_fix_stack_asms still have equal part.
It's a bug.

Denis.



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