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: [patch] fr30: Hookize some target macros.


Kazu Hirata wrote:
The patch has one caveat.  fr30_setup_incoming_varargs() has one line
of code #if 0'ed by STRICT_ARGUMENT_NAMING.  I replaced with a
run-time check:

The code has been there effectively unchanged since it was contributed to the FSF. If there was ever any reason for doing this, it is probably in the Red Hat CVS tree, and only a Red Hat employee can check that.


You could at least add a ??? comment pointing out that the run-time test for strict_argument_naming seems to be unnecessary.

Built cc1 of fr30-elf. OK to apply?

OK. -- Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com


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