From: mjn3@codepoet.org (Manuel Novoa III)
To: Riley Williams <Riley@Williams.Name>
Cc: Robert de Bath <robert$@mayday.cix.co.uk>,
Linux-8086 <linux-8086@vger.kernel.org>
Subject: Re: More dev86 changes (0.16.5)
Date: Thu, 25 Jul 2002 09:42:19 -0600 [thread overview]
Message-ID: <20020725154219.GA2577@codepoet.org> (raw)
In-Reply-To: <Pine.LNX.4.21.0207242221470.30515-100000@Consulate.UFP.CX>
Robert and Riley,
First, thanks Robert for all the work you've done in integrating
and improving these changes. It will certainly make my work easier.
On Wed, Jul 24, 2002 at 11:02:44PM +0100, Riley Williams wrote:
> >> 1) #elif support.
>
> > In 0.16.5
> > Do we want #elifdef too?
>
> If we support it as a variant of #if then we also want it as a variant
> of #elif as well, in my opinion. Therefore, we will want...
>
> #elif
> #elifdef
> #elifndef
>
> ...together with any other variants that may exist.
> Also (inspired by the way it's used in your example below, but taken
> from a C compiler I once used on a Burroughs B6700 mainframe), could
> support be added for the following...
>
> a. #debug msg
> b. #debugn msg
> c. #info msg
> d. #onerr msg
#elif is C89. While the others could be convenient at times, I know
that I wouldn't use them for portability reasons.
> I would like to see the code produe a warning for the fact that the
> macro defenition for strong_alias() includes the terminating semicolon
> that should never be present on macro definitions - and yes, I know the
> Linux kernel uses several macros that break this rule. Linus himself
> states that they're wrong, but they're so well embedded that it's more
> hassle than it's worth to fix the problem...
The trailing semicolon was strictly a convenience thing. The uClibc
code contains several instances of strong_alias(). Generally where
it is used, there is no trailing semicolon. This caused a problem
when building with bcc since, although the patch allowed asm() to be
used at file scope, it was still a statement and needed trailing
semicolon for bcc to be happy. Rather than change all the offending
instances in uClibc at the time, I just put the ';' in the bcc macro
definition. I've got no problem changing this before the port is
committed.
> One other thing I would like to see, which is an optimising measure more
> than anything: When bcc lays out the variables at any level, if it sorts
> them in descending order of size before laying them out, it will really
> minimise the number of padding bytes needed to ensure correct alignment.
> It is for this reason that when I am writing C code, I always list all
> of the variables in descending size order,
Same here. ;-)
> but the compiler could make
> this irrelevant, as many other optimising measures have become totally
> irrelevant over the years.
Manuel
next prev parent reply other threads:[~2002-07-25 15:42 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-07-14 14:05 C compiler, assembler and linker Harry Kalogirou
2002-07-14 17:27 ` Manuel Novoa III
2002-07-15 6:07 ` Riley Williams
2002-07-15 17:02 ` Manuel Novoa III
2002-07-15 19:13 ` Riley Williams
2002-07-15 22:02 ` Manuel Novoa III
2002-07-16 6:27 ` Riley Williams
2002-07-17 1:31 ` Manuel Novoa III
2002-07-17 6:33 ` Riley Williams
2002-07-18 12:03 ` Minix vs. ELKS Feher Tamas
2002-07-18 12:27 ` Javier Sedano
2002-07-02 15:07 ` (unknown) Miguel A. Bolanos
2002-07-18 13:40 ` Minix vs. ELKS Alan Cox
2002-07-22 21:41 ` C compiler, assembler and linker Robert de Bath
2002-07-23 8:16 ` Robert de Bath
2002-07-23 16:25 ` Manuel Novoa III
2002-07-23 19:09 ` Robert de Bath
2002-07-24 21:17 ` More dev86 changes (0.16.5) Robert de Bath
2002-07-24 22:02 ` Riley Williams
2002-07-25 15:42 ` Manuel Novoa III [this message]
2002-07-26 7:55 ` Robert de Bath
2002-07-26 15:12 ` Manuel Novoa III
2002-07-26 8:22 ` Robert de Bath
2002-07-24 22:26 ` Paul Nasrat
2002-07-25 16:34 ` Manuel Novoa III
2002-07-22 23:26 ` C compiler, assembler and linker Robert de Bath
2002-07-23 0:34 ` Riley Williams
2002-07-23 0:58 ` Manuel Novoa III
2002-07-23 0:46 ` Manuel Novoa III
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20020725154219.GA2577@codepoet.org \
--to=mjn3@codepoet.org \
--cc=Riley@Williams.Name \
--cc=linux-8086@vger.kernel.org \
--cc=robert$@mayday.cix.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox