From: Stafford Horne <shorne@gmail.com>
To: gdb-patches@sourceware.org
Cc: openrisc@lists.librecores.org,
Stefan Wallentowitz <stefan@wallentowitz.de>
Subject: [PATCH 12/18] gdb: or1k: Apply changes of the last two years
Date: Wed, 23 Nov 2016 22:13:00 -0000 [thread overview]
Message-ID: <1479939044-1341-13-git-send-email-shorne@gmail.com> (raw)
In-Reply-To: <1479939044-1341-1-git-send-email-shorne@gmail.com>
From: Stefan Wallentowitz <stefan@wallentowitz.de>
gdb/ChangeLog:
* or1k-tdep.c (or1k_analyse_inst): s/fatal/throw_quit/
(or1k_skip_prologue): use SYMTAB_COMPUNIT and
COMPUNIT_DEBUGFORMAT
(or1k_frame_cache): s/fatal/throw_quit/
(or1k_regset_from_core_section): remove
(or1k_iterate_over_regset_sections): create, blank
implementation
---
gdb/or1k-tdep.c | 44 ++++++++++++++++----------------------------
1 file changed, 16 insertions(+), 28 deletions(-)
diff --git a/gdb/or1k-tdep.c b/gdb/or1k-tdep.c
index ae57734..8cfa440 100644
--- a/gdb/or1k-tdep.c
+++ b/gdb/or1k-tdep.c
@@ -89,7 +89,7 @@
#include "demangle.h"
#include "defs.h"
-#include "gdb_string.h"
+#include <string.h>
#include "frame.h"
#include "inferior.h"
#include "symtab.h"
@@ -260,7 +260,7 @@ or1k_analyse_inst (uint32_t inst,
/* Check we got something, and if so skip on */
if (start_ptr == end_ptr)
{
- fatal ("bitstring \"%s\" at offset %d has no length field.\n",
+ throw_quit ("bitstring \"%s\" at offset %d has no length field.\n",
format, i);
}
@@ -271,7 +271,7 @@ or1k_analyse_inst (uint32_t inst,
just should not be wrong. */
if ('b' != format[i++])
{
- fatal ("bitstring \"%s\" at offset %d has no terminating 'b'.\n",
+ throw_quit ("bitstring \"%s\" at offset %d has no terminating 'b'.\n",
format, i);
}
@@ -292,7 +292,7 @@ or1k_analyse_inst (uint32_t inst,
break;
default:
- fatal ("invalid character in bitstring \"%s\" at offset %d.\n",
+ throw_quit ("invalid character in bitstring \"%s\" at offset %d.\n",
format, i);
break;
}
@@ -977,7 +977,8 @@ or1k_skip_prologue (struct gdbarch *gdbarch,
if (0 != prologue_end)
{
struct symtab_and_line prologue_sal = find_pc_line (start_pc, 0);
- const char *debug_format = prologue_sal.symtab->debugformat;
+ struct compunit_symtab *compunit = SYMTAB_COMPUNIT(prologue_sal.symtab);
+ const char *debug_format = COMPUNIT_DEBUGFORMAT(compunit);
if ((NULL != debug_format) && (strlen ("dwarf") <= strlen (debug_format))
&& (0 == strncasecmp ("dwarf", debug_format, strlen ("dwarf"))))
@@ -1575,7 +1576,7 @@ or1k_frame_cache (struct frame_info *this_frame,
are frameless and can give up here. */
if (end_addr < start_addr)
{
- fatal ("end addr 0x%08x is less than start addr 0x%08x\n",
+ throw_quit ("end addr 0x%08x is less than start addr 0x%08x\n",
(unsigned int) end_addr, (unsigned int) start_addr);
}
@@ -1850,28 +1851,15 @@ or1k_frame_base_sniffer (struct frame_info *this_frame)
} /* or1k_frame_base_sniffer () */
#endif
-
-/* -------------------------------------------------------------------------- */
-/*!Return information needed to handle a core file.
-
- We put together a regset structure that tells the system how to transfer
- registers to and from a core file image.
-
- @param[in] gdbarch The GDB architecture we are using.
- @param[in] sect_name The name of the section being considered.
- @param[in] sect_size The size of the section being considered.
-
- @return A regset structure for the section, or NULL if none is available. */
-/* -------------------------------------------------------------------------- */
-static const struct regset *
-or1k_regset_from_core_section (struct gdbarch *gdbarch,
- const char *sect_name,
- size_t sect_size)
+/* Iterate over core file register note sections. */
+static void
+or1k_iterate_over_regset_sections (struct gdbarch *gdbarch,
+ iterate_over_regset_sections_cb *cb,
+ void *cb_data,
+ const struct regcache *regcache)
{
- printf ("sect_name \"%s\", sect_size %zd\n", sect_name, sect_size);
- return NULL;
-
-} /* or1k_regset_from_core_section () */
+ // TODO: Do we need to put something here? (wallento)
+}
/* -------------------------------------------------------------------------- */
/*!Create a register group based on a group name.
@@ -2091,7 +2079,7 @@ or1k_gdbarch_init (struct gdbarch_info info,
#endif
/* Handle core files */
- set_gdbarch_regset_from_core_section (gdbarch, or1k_regset_from_core_section);
+ set_gdbarch_iterate_over_regset_sections (gdbarch, or1k_iterate_over_regset_sections);
/* Frame unwinders. Use DWARF debug info if available, otherwise use our
own unwinder. */
--
2.7.4
next prev parent reply other threads:[~2016-11-23 22:11 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-23 22:11 [PATCH 00/18] gdb: Port for OpenRISC Stafford Horne
2016-11-23 22:11 ` [PATCH 09/18] gdb: or1k: Silence gcc warning Stafford Horne
2016-11-23 22:11 ` [PATCH 03/18] gdb: or1k: Add or1k_push_dummy_code and use it Stafford Horne
2016-11-23 22:11 ` [PATCH 05/18] gdb: handle arbitrary strings in tdesc_register_in_reggroup_p Stafford Horne
2016-11-23 22:11 ` [PATCH 15/18] gdb: or1k: Update calling conventions to help pass tests Stafford Horne
2016-11-23 22:11 ` [PATCH 16/18] gdb: Remove corelow object from or1k linux build Stafford Horne
2016-11-23 22:11 ` [PATCH 04/18] gdb: add tdesc_find_register functions Stafford Horne
2016-11-23 22:11 ` [PATCH 17/18] gdb: or1k: Updates for compiler type checking Stafford Horne
2016-11-23 22:11 ` [PATCH 13/18] gdb: Add OpenRISC to texinfo menu Stafford Horne
2016-11-23 22:13 ` [PATCH 08/18] gdb: or1k: change or1k_push_dummy_code stack alignment Stafford Horne
2016-11-23 22:13 ` [PATCH 10/18] gdb: or1k: Fixes for things that broke after upstream merge Stafford Horne
2016-11-23 22:13 ` Stafford Horne [this message]
2016-11-23 22:13 ` [PATCH 01/18] gdb: add or1k and or1knd target support Stafford Horne
2016-11-23 22:14 ` [PATCH 06/18] gdb: or1k: set gdbarch num regs to gpr + spr regs Stafford Horne
2016-11-23 22:14 ` [PATCH 14/18] gdb: testsuite: Add or1k l.nop inscruction Stafford Horne
2016-11-23 22:14 ` [PATCH 07/18] gdb: or1k: add target descriptor support Stafford Horne
2016-11-23 22:16 ` [PATCH 02/18] gdb: or1k: Add a check for NULL ptr in or1k_skip_prologue Stafford Horne
2016-11-23 22:16 ` [PATCH 11/18] Add gdb for or1k build Stafford Horne
2016-11-23 22:23 ` [PATCH 18/18] gdb: or1k: Refactor to new bp_kind_from_pc and pb_from_kind Stafford Horne
2016-11-24 20:37 ` [PATCH 00/18] gdb: Port for OpenRISC Yao Qi
2016-11-25 22:50 ` Stafford Horne
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=1479939044-1341-13-git-send-email-shorne@gmail.com \
--to=shorne@gmail.com \
--cc=gdb-patches@sourceware.org \
--cc=openrisc@lists.librecores.org \
--cc=stefan@wallentowitz.de \
/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