Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Wu Zhou" <woodzltc@cn.ibm.com>
To: "Daniel Jacobowitz" <drow@false.org>
Cc: <gdb-patches@sources.redhat.com>
Subject: Re: about how to add support to new c++ compiler in GDB
Date: Fri, 29 Apr 2005 02:37:00 -0000	[thread overview]
Message-ID: <012801c54c64$5b02d500$4186b509@ibmcsdl9m89c83> (raw)
In-Reply-To: <20050428131902.GB29277@nevyn.them.org>

> Could you please respond to the list in the future?  Thanks in advance!

Sorry, I forgeted to cc the mail-list. Ever since I will do that as you
request.

> >
> > Daniel, I noticed that you also include some patches to completion.exp,
> > volatile.exp and gdb1355.exp. That is also what I wan to include.
Besides
> > these "long" or "long int", "unsigned short" or "short unsigned"
problem, I
> > also encounter one problem with "signed" while running volatile.exp with
xlc
> > compiler. xlc compiler take "signed" as the default. so in the outputed
> > debuginfo of function qux2:
> >
> > qux2 (volatile unsigned char vuc, const volatile int cvi,
> >       volatile short /*&*/vsr, volatile long *vlp, float *volatile fpv,
> >       const volatile signed char *const volatile cvscpcv)
> >
> > cvscpcv will be interpreted as "const volatile char *const volatile". Do
you
> > think that it is ok to also add a bracket around "signed" to let this
pass?
> > Thanks.
>
> Er... that is not OK.  If necessary we can add an XFAIL for xlc,
> though.  This is a bug in xlc; "signed char" and "char, defaulting to
> signed behavior" are not the same types in C.
>

OK. Maybe I could try to convince XLC guys to fix this first.

> > BTW: do you means that you sill have another 40(41 - 1) patches for RVCT
and
> > that this patch is only the first one? If it is like that, I am really
> > curious about how many codes you will add into GDB. :-)
>
> No, this is all 41 rolled up into one big diff :-)

Then, which part will get your first attention? I suggest you start with
eliminating the dependence on DW_TAG_containing_type first. Then XLC guys
could go on improving their debuginfo output. What is your point on this?
Please comment. Thanks.

BTW. I had verify your patch, It did worked. No SEGV error any more.
Althought there are still some error, most of them should be XLC specific I
believe. I could work with XLC guys to improve them.

Cheers
- Wu Zhou




  parent reply	other threads:[~2005-04-29  2:37 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-26  2:54 Wu Zhou
2005-04-26  3:18 ` Daniel Jacobowitz
2005-04-26  9:53   ` Wu Zhou
2005-04-26 13:20     ` Daniel Jacobowitz
2005-04-27 20:09   ` Daniel Jacobowitz
     [not found]     ` <019101c54bc9$20360cd0$4186b509@ibmcsdl9m89c83>
     [not found]       ` <20050428131902.GB29277@nevyn.them.org>
2005-04-29  2:37         ` Wu Zhou [this message]
2005-04-29  7:28           ` Wu Zhou
2005-04-29 13:10             ` Daniel Jacobowitz
2005-04-28  6:34 woodzltc
2005-05-09 12:42 Wu Zhou

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='012801c54c64$5b02d500$4186b509@ibmcsdl9m89c83' \
    --to=woodzltc@cn.ibm.com \
    --cc=drow@false.org \
    --cc=gdb-patches@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