public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: "Casper Hornstrup" <chorns@users.sourceforge.net>
To: "'Fergus Henderson'" <fjh@cs.mu.OZ.AU>
Cc: <gcc-patches@gcc.gnu.org>
Subject: RE: [PATCH] Fastcall support on cygwin and mingw targets
Date: Tue, 12 Nov 2002 10:45:00 -0000	[thread overview]
Message-ID: <002e01c28a7b$9f60ffe0$0300000a@csiteoffice.csite.com> (raw)
In-Reply-To: <20021111192817.GA13166@ceres.cs.mu.oz.au>



> -----Original Message-----
> From: Fergus Henderson [mailto:fjh@ceres.cs.mu.OZ.AU] On 
> Behalf Of Fergus Henderson
> Sent: 11. november 2002 20:28
> To: Casper Hornstrup
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Fastcall support on cygwin and mingw targets
> 
> 
> On 11-Nov-2002, Casper Hornstrup <chorns@users.sourceforge.net> wrote:
> > 
> > > What happens if a function has multiple attributes,
> > > e.g. both fastcall and stdcall or both fastcall and
> > > regparm(3)? It may be worth adding a test for that case.
> > > 
> > 
> > stdcall attribute after fastcall attribute - fastcall calling 
> > convention is used, but the symbol is prefixed with _ not @.
> > 
> > Obviously a bug, but how should this situation be handled? Which 
> > attribute should win and should gcc emit a warning or error out?
> >
> > fastcall attribute after stdcall attribute - same result. fastcall 
> > attribute after regparm(3) attribute - fastcall wins.
> > regparm(3) attribute after fastcall attribute - fastcall wins.
> 
> IMHO the desirable behaviour is for gcc to report an error
> in all of these cases.

Where/how should this check be performed? It can't be done in the
attribute handler since all attributes may not be added to the
attribute list yet.

> 
> > > >+ The @code{fastcall} calling convention exists in 
> Windows NT for 
> > > >+ Intel
> > > >+ processors only.
> > > 
> > > That part of the documentation is wrong.
> > > Cygwin and Mingw work on most variants of Windows, not just
> > > Windows NT.
> > 
> > I think it is a reference to fastcall only being used on 
> Intel Windows 
> > NT platforms (eg. it was not used on Windows NT/Alpha). Maybe it 
> > should be removed?
> 
> Either that, or just delete the "NT".

I'll delete "NT".

  reply	other threads:[~2002-11-12 18:45 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-30 21:14 PATCH: Re: optimization/6162: gcc 3.0.4: certain i386 asm reloader ice Nathanael Nerode
2002-11-10  8:17 ` [PATCH] Fastcall support on cygwin and mingw targets Casper Hornstrup
2002-11-10 17:50   ` Fergus Henderson
2002-11-10 19:10     ` Christopher Faylor
2002-11-11 12:47     ` Casper Hornstrup
2002-11-11 12:27       ` Fergus Henderson
2002-11-12 10:45         ` Casper Hornstrup [this message]
2002-11-12 22:08           ` Fergus Henderson
2002-11-13 12:17             ` Casper Hornstrup
2002-11-13 15:47               ` Joseph S. Myers
2002-11-13 20:27               ` Fergus Henderson
2002-11-14  0:31                 ` Joseph S. Myers
2002-11-14  8:28                 ` Casper Hornstrup
2002-11-25 15:39                   ` Richard Henderson
2002-11-27 19:52                     ` Casper Hornstrup
2002-11-27 21:15                       ` Casper Hornstrup
2002-11-27 22:19                       ` Richard Henderson
2002-11-28  2:00                         ` Richard Henderson
2002-12-02 13:25                     ` Casper Hornstrup
2002-12-02 13:48                       ` Richard Henderson
2002-12-03 13:52                         ` Casper Hornstrup
2002-12-03 16:32                           ` Richard Henderson
2002-12-08  4:08                             ` Casper Hornstrup
2002-12-19 14:01                               ` Richard Henderson
2002-12-19 14:07                                 ` Casper Hornstrup
2002-11-20 11:03 Casper Hornstrup

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='002e01c28a7b$9f60ffe0$0300000a@csiteoffice.csite.com' \
    --to=chorns@users.sourceforge.net \
    --cc=fjh@cs.mu.OZ.AU \
    --cc=gcc-patches@gcc.gnu.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).