Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: mec.gnu@mindspring.com (Michael Elizabeth Chastain)
To: ezannoni@redhat.com, gdb@sources.redhat.com, jimb@redhat.com
Subject: gcc HEAD, stabs+, TYPE_CODE_INT problem
Date: Mon, 05 Apr 2004 19:09:00 -0000	[thread overview]
Message-ID: <20040405190942.23D084B104@berman.michael-chastain.com> (raw)

My last test run got stabbed really badly by a new problem
with gcc HEAD -gstabs+.  There are about 700 new non-PASS results.

The symptoms are: look at a global variable of type "char *".

  # gcc 3.3.3
  (gdb) ptype s
  type = char *

  # gcc HEAD 2004-03-30
  (gdb) ptype s
  type = <invalid type code 7> *

"invalid type code 7" is TYPE_CODE_INT.

With gcc 3.3.3, the stabs look like this:

  # gcc 3.3.3
  .stabs	"char:t(0,2)=r(0,2);0;127;",128,0,0,0
  .stabs	"__caddr_t:t(7,35)=(7,36)=*(0,2)",128,0,82,0
  .stabs	"s:G(7,36)",32,0,7,0

With gcc HEAD, the stabs look like this:

  # gcc HEAD
  .stabs	"__caddr_t:t(3,44)=(3,45)=*(3,46)=r(3,46);0;127;",128,0,82,0
  .stabs	"s:G(3,45)",32,0,7,0

That is, gcc 3.3.3 emits a separate line for each primitive
type such as "char".  gcc HEAD emits the definition of "char"
as a nested definition inside the first type that uses char,
such as pointer-to-char.

The big question is: is this legal stabs?  After reading
stabs.texinfo, I'm inclined to think that it is.

If it's legal stabs, then someone has to enhance the stab reader.  I
haven't started debugging gdb yet but I suspect that bit of code is
getting confused by the new-style nested definition.  There's a bit of
code in read_range_type to recognize the special case of "char" as 0 to
127, and gdb is behaving like that special-case isn't getting
recognized.

If this code is *not* legal stabs, then I can make a small test case
and then file a gcc PR.  I already isolated the gcc patch which
causes this.  It's a big C-declaration rewrite.

Michael C


             reply	other threads:[~2004-04-05 19:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-05 19:09 Michael Elizabeth Chastain [this message]
2004-04-05 19:32 ` Daniel Jacobowitz
2004-04-05 20:26 Michael Elizabeth Chastain
2004-04-05 20:47 ` Zack Weinberg
2004-04-05 21:36 Michael Elizabeth Chastain
2004-04-05 23:26 ` Zack Weinberg

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=20040405190942.23D084B104@berman.michael-chastain.com \
    --to=mec.gnu@mindspring.com \
    --cc=ezannoni@redhat.com \
    --cc=gdb@sources.redhat.com \
    --cc=jimb@redhat.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