Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: "Gary Funck" <gary@intrepid.com>
To: <gdb@sourceware.org>
Cc: "'Jim Wilson'" <wilson@specifix.com>
Subject: RE: how to support C type qualifiers applied to arrays?
Date: Tue, 09 Jan 2007 17:01:00 -0000	[thread overview]
Message-ID: <000501c7340f$d4c8ed40$0a0a0a0a@DELORIAN> (raw)
In-Reply-To: <1167689747.2505.17.camel@localhost.localdomain>

Jim Wilson wrote (in part):
> This seems to answer the question I asked long ago.  Why are we trying
> to put back qualifiers from the decl?  Because gen_type_die stripped
> them off.  This seems wrong.
>

It seems that the use of type_main_variant()
dates back to the original port of the DWARF 1 generator
to the DWARF 2 implementation.  I didn't try understanding the
DWARF 1 generator in detail, but it seemed that the use of the
type's main variant helped limit the number of unique type dies
that were generated.  In any event, it doesn't appear that there
is a compelling reason to continue to use the type_main_variant()
for DWARF 2 generatiion, especially in light of the fact that
in some cases the DWARF 2 information is incorrect in the current
implmentation.

There are of course the risks that you mentioned
in your reply that there may now be implicit assumptions made
by the various clients (C, C++, Fortran, Ada, for example)
regarding dependencies upon the type_main_variant(), but we
may have to deal with those issues as they arise.
 
> If we fix gen_type_die to stop calling type_main_variant, and 
> if we fix
> gen_variable_die to stop adding back the type qualifiers, 
> then I get the
> right result.  So I think I was on the right track before, we 
> just need
> another little change to gen_type_die in addition to what I already
> described.

I'll give this a try, and follow up.


      reply	other threads:[~2007-01-09 17:01 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-27 22:42 Gary Funck
2006-11-27 23:10 ` Daniel Jacobowitz
2006-11-27 23:30   ` Jim Blandy
2006-11-27 23:52     ` Daniel Jacobowitz
2006-11-27 23:45   ` Joseph S. Myers
2006-11-27 23:55     ` Daniel Jacobowitz
2006-12-14 20:22       ` Gary Funck
2006-12-15 22:36         ` Jim Blandy
2006-12-19 19:09           ` Gary Funck
2006-12-20  2:40           ` Gary Funck
2006-12-27  2:12             ` Jim Blandy
2007-01-01 22:16         ` Jim Wilson
2007-01-09 17:01           ` Gary Funck [this message]

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='000501c7340f$d4c8ed40$0a0a0a0a@DELORIAN' \
    --to=gary@intrepid.com \
    --cc=gdb@sourceware.org \
    --cc=wilson@specifix.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