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]

Re: FIX_REGISTER_CHECK


The patch seems incomplete, in two different ways.
1) You are adding an error message for invalid -fcall* options, but you
   only do so for two registers.  You don't give errors for all invalid
   options.
2) You are adding an error message for invalid -fcall* options, but you
   only do so for one target.  You don't give errors for all targets.
Both of these problems seem undesirable to me.

If it is really only the stack pointer and frame pointer that you are worried
about at this time, then why make the patch MIPS specific?  These registers
would be invalid for all targets.  It would make the patch simpler to check
them for every target.  This would avoid the need for a new machine dependent
macro, and eliminates one of my objections, and gives a good argument against
the other.

Jim


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