public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Paul Koning <pkoning@equallogic.com>
To: rth@redhat.com
Cc: ian@airs.com, pinskia@physics.uc.edu, mrs@apple.com,
	dave.korn@artimi.com, gcc@sources.redhat.com, sjackman@gmail.com
Subject: Re: memcpy to an unaligned address
Date: Thu, 04 Aug 2005 12:40:00 -0000	[thread overview]
Message-ID: <17138.3224.786605.462292@gargle.gargle.HOWL> (raw)
In-Reply-To: <20050803180011.GA16808@redhat.com>

>>>>> "Richard" == Richard Henderson <rth@redhat.com> writes:

 >> > No it is not, once you take the address (which should be
 >> rejected), it > is of type "unsigned int *" and not unaligned
 >> variable, passing it to > memcpy assumes the type alignment is the
 >> natural alignment.
 >> 
 >> That argument doesn't make sense to me.

 Richard> It is nevertheless correct.  Examine all of the parts of the
 Richard> expression.

 Richard> In particular, "&s->b".  What type does it have?  In an
 Richard> ideal world, it would be "pointer to unaligned integer".
 Richard> But we have no such type in our type system, so it is
 Richard> "pointer to integer".  This expression is ONLY THEN passed
 Richard> to memcpy.  At which point we query the argument for its
 Richard> alignment, and get the non-intuitive result.

The underlying problem is that the type system in GCC isn't right.

The C type system has data of various kinds, pointers to them,
structures made up of the above, etc.  GCC extends the type system by
introducing selectable alignment.  But it doesn't do it consistently.
We have int, we have *int, we have packed int, but we don't have 
*packed int.

So the outcome is "correct" only if you treat GCC's incomplete type
system as correct, which I don't agree with.  This does mean,
unfortunately, that the fix is to correct that incompleteness, which
was said to be a nontrivial task.

    paul

  parent reply	other threads:[~2005-08-04 12:40 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-02 17:32 Shaun Jackman
2005-08-02 17:43 ` Dave Korn
2005-08-02 19:13   ` Shaun Jackman
2005-08-02 19:19     ` Paul Koning
2005-08-02 19:26       ` Shaun Jackman
2005-08-02 19:40         ` Dave Korn
2005-08-02 19:48           ` Paul Koning
2005-08-02 20:15           ` Shaun Jackman
2005-08-02 20:29             ` Mike Stump
2005-08-02 20:38               ` Andrew Pinski
2005-08-02 20:45                 ` Ian Lance Taylor
2005-08-02 21:30                   ` Mike Stump
2005-08-02 21:34                     ` Joe Buck
2005-08-03 18:00                   ` Richard Henderson
2005-08-03 18:15                     ` Shaun Jackman
2005-08-03 18:19                       ` Dave Korn
2005-08-03 21:26                       ` Richard Henderson
2005-08-04  4:42                     ` Ian Lance Taylor
2005-08-04 12:40                     ` Paul Koning [this message]
2005-08-02 20:46                 ` Paul Koning
2005-08-02 22:17                   ` Shaun Jackman
2005-08-03 17:16                     ` Paul Koning
2005-08-02 22:26                   ` Shaun Jackman
2005-08-02 22:29                     ` Shaun Jackman
2005-08-02 21:05                 ` Mike Stump
2005-08-02 21:11                   ` Joe Buck
2005-08-02 22:15                     ` Shaun Jackman
2005-08-02 22:12                       ` Joe Buck
2005-08-02 20:29             ` Paul Koning
2005-08-02 17:48 ` Falk Hueffner
2005-08-02 18:03 ` Mike Stump
     [not found] <345be691050804025955c0b4ab@mail.gmail.com>
2005-08-04 15:06 ` Shaun Jackman
2005-08-04 15:09   ` Christian Joensson
2005-08-05  8:41   ` Carl Whitwell
2005-08-05 16:09     ` Shaun Jackman

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=17138.3224.786605.462292@gargle.gargle.HOWL \
    --to=pkoning@equallogic.com \
    --cc=dave.korn@artimi.com \
    --cc=gcc@sources.redhat.com \
    --cc=ian@airs.com \
    --cc=mrs@apple.com \
    --cc=pinskia@physics.uc.edu \
    --cc=rth@redhat.com \
    --cc=sjackman@gmail.com \
    /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).