Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Randolph Chung <randolph@tausq.org>
To: gdb@sources.redhat.com
Subject: RFC: getting rid of deprecated_hp_som_som_object_present
Date: Wed, 01 Mar 2006 15:12:00 -0000	[thread overview]
Message-ID: <4405B9C0.90505@tausq.org> (raw)

Recently on gdb-patches there was some mention of this deprecated flag. 
I'd like to get some comments/feedback on how to clean this up....

Here are the parts of gdb that uses this flag:

- symtab.c defines this variable
- hppa-hpux-tdep.c initializes this to 0 for each newly created inferior
it also uses this to help determine if we should use the special hp c++ 
exception support
- hpread.c sets this variable to 1 if we read a file with HP debug 
information
- eval.c uses this to decide to punt if we try to do something with a 
pointer to member function, or to apply a bias for a pointer to member 
variable
- parse.c has a reference to this in a function named 
parse_nested_classes_for_hpacc, but this code apparently is no longer 
used anywhere in gdb
- valops.c uses this in value_cast to handle casting things from a 
pointer to member function/variable to an int or enum
- c-typeprint.c uses this to print extra info for "sized enums"
- cp-valprint.c uses this in cp_print_class_method to handle class 
member functions (I think)

Most of these have to do with how gdb handles c++ methods and member 
variables, which makes me think that perhaps we should add some hooks to 
the cp-abi layer to handle the type conversion logic. This takes care of 
eval.c, valops.c, and maybe cp-valprint.c

I'll have to dig through some cvs history to figure out what we want to 
do with the bits in parse.c

hpread.c should probably just set the current auto c++ abi type.

what should we do with c-typeprint.c?

I notice that HP's wdb has better support for some of these cases. If we 
were to clean up the above it will be worthwhile to look through wdb and 
try to merge in some of the logic there (mostly to deal with member 
functions)

Any thoughts? Is this even worth touching/fixing?

randolph


             reply	other threads:[~2006-03-01 15:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-01 15:12 Randolph Chung [this message]
2006-03-02  5:57 ` Jim Blandy
2006-08-02  2:55 ` Is anyone using the HP compilers on PA-RISC with FSF GDB? Daniel Jacobowitz
2006-08-04  5:00   ` Albert Chin
2006-08-04 12:50     ` Daniel Jacobowitz
2006-08-11 15:54       ` Albert Chin

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=4405B9C0.90505@tausq.org \
    --to=randolph@tausq.org \
    --cc=gdb@sources.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