Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <cagney@gnu.org>
To: Sherry Samuel <SherryS@KPITCummins.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [Patch H8/300]  : Fix for  crashing of gdb with" info f" command
Date: Thu, 15 Jul 2004 18:18:00 -0000	[thread overview]
Message-ID: <40F6CA8F.5000504@gnu.org> (raw)
In-Reply-To: <4A1BE23A7B777442B60F4B4916AE0F130218AAA5@sohm.kpit.com>

> Hi
> Reference:http://sources.redhat.com/ml/gdb-patches/2004-06/msg00428.html
> 
> In the latest GDB(gdb+dejagnu-20040630) snap shot, the "info f" problem described in the above link is fixed. Referring through the links you have suggested, it is evident that h8300 target is not in the list of the new GDB frame structure. Is it possible to know what should be done from our side to update H8300 targets to the new frame work? Can you suggest how to go about it?
> 
> Thanks in advance.

Right,

A very simplistic cookbook is as follows:

- #if 0 all the set_gdbarch_deprecated ... lines
- build gdb
- try gdb on a simple program
   break main, info registers, stepi, nexti, ...
- fix an internal error due to missing method
- repeat

eventually you'll get gdb kind of working, at that point repeat the 
above using the testsuite.

It turns out that for architectures like this that use all the old 
deprecated methods, its quicker / easier to clear the deck and implement 
a new (borrowing heavily from the old code of course).

It also turns out that basic operations (break main; run, stepi, info 
registers) don't even need a frame unwinder - so that can be left as a 
separate pass.

For the unwinder, you need to convert the frame_saved_regs code into the 
frame_unwind_cache code (seen in more up-to-date architectures).

As for contributing this, perhaphs we can just incrementally submit the 
above in realtime?

Andrew



  reply	other threads:[~2004-07-15 18:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-09 12:12 Sherry Samuel
2004-07-15 18:18 ` Andrew Cagney [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-06-18 10:35 Sherry Samuel
2004-06-22 20:09 ` Andrew Cagney

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=40F6CA8F.5000504@gnu.org \
    --to=cagney@gnu.org \
    --cc=SherryS@KPITCummins.com \
    --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