This is the mail archive of the
mailing list for the GCC project.
Re: new port: m32c
- From: DJ Delorie <dj at redhat dot com>
- To: geoffk at apple dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 20 Jul 2005 17:27:28 -0400
- Subject: Re: new port: m32c
- References: <140A7AB8-4B53-4771-B4B9-886E722C2805@apple.com>
> > /*#define FIRST_PSEUDO_REGISTER 16*/
> > #define FIRST_PSEUDO_REGISTER 20
> Why the commented-out code? Likewise,
It's already gone. http://people.redhat.com/~dj/
> and a number of other places. I would suggest searching for every
> '#if 0' in the port and either deleting the code or replacing it with
> '#if DEBUG' or similar.
There's a couple of chunks of if-0 code that, in theory, *should* work
but at the moment *don't* work. I suppose I should put in a comment
as to why I'm keeping them around (some already have such a comment).
Most of the rest are single-use debug chunks. The ones that are
useful as a group are already "if DEBUG_*" but most of them are ones I
enable singly. I don't like having to continuously re-add printfs,
but I almost never turn them all on at once, so a global DEBUG would
I suppose I could use '#if DEBUG0' and '#if DEBUG1' for tags, so I can
easily toggle them on/off yet still have descriptive logic. Would
that be acceptable?
> - Comments. Please add a comment to the top of every routine in
> - Strange indenting
Blame indent! I ran everything through indent before submitting it,
and it did strange things to the code. I've since fixed the obvious
> I'd appreciate it if you could use MIME, rather than UUEncode, for
> future patches. The web archives know about MIME but don't know how
> to uudecode.
I have no convenient way of attaching it as a MIME attachment.
Neither my mailer nor command line tools know how to do that. If this
is really important, I can install yet another mailer just to send out
these patches, but it seems to me that the archives should be told how
to deal with uuencoded attachments. Meanwhile, I've been putting the
current patchset on the web instead of filling up the archives with
huge repeat posts. Obviously, I'll post an "as committed" patch when