patch: morpho ms1-elf support
Paul Brook
paul@codesourcery.com
Tue Jul 5 21:13:00 GMT 2005
On Tuesday 05 July 2005 21:47, DJ Delorie wrote:
> > (in some cases, "MS1 implementation of target macro FOO" may be
> > sufficient comment, leaving it to the internals manual to detail the
> > parameters and return value of that macro).
>
> Can we please not entranch this silliness? If the function is
> obviously an implementation of a documented macro, and does what the
> manual says it does, let's not waste time saying "this does what you
> expect". Let's spend that time fixing real bugs instead.
I agree in principle. However in my experience you can usually say something
meaningful about most target macros. eg.
/* MS1 does not have FOO */
#define TARGET_FOO 0
IMHO this is a useful comment even though it doesn't tell men anything
directly. The presence of a "this is obvious" comment implies that I'm not
missing something subtle, and that the original author of the port has
considered the value, and wasn't just cut/pasting or making values up at
random[1].
Paul
[1] This is a general statement, not an assessment of the quality of this
particular patch :-)
More information about the Gcc-patches
mailing list