[PATCH 01/33] gdb: add getter/setter for compunit_symtab::objfile
Simon Marchi
simon.marchi@polymtl.ca
Fri Jan 28 12:44:59 GMT 2022
From: Simon Marchi <simon.marchi@efficios.com>
Rename the field to m_objfile, and add a getter and a setter. Update
all users.
Change-Id: If7e2f763ee3e70570140d9af9261b1b056253317
---
gdb/symfile.c | 8 ++++----
gdb/symmisc.c | 2 +-
gdb/symtab.c | 2 +-
gdb/symtab.h | 14 ++++++++++++--
4 files changed, 18 insertions(+), 8 deletions(-)
diff --git a/gdb/symfile.c b/gdb/symfile.c
index 9bd4f48d0b02..a660422b51e0 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -2774,7 +2774,7 @@ deduce_language_from_filename (const char *filename)
struct symtab *
allocate_symtab (struct compunit_symtab *cust, const char *filename)
{
- struct objfile *objfile = cust->objfile;
+ struct objfile *objfile = cust->objfile ();
struct symtab *symtab
= OBSTACK_ZALLOC (&objfile->objfile_obstack, struct symtab);
@@ -2832,7 +2832,7 @@ allocate_compunit_symtab (struct objfile *objfile, const char *name)
struct compunit_symtab);
const char *saved_name;
- cu->objfile = objfile;
+ cu->set_objfile (objfile);
/* The name we record here is only for display/debugging purposes.
Just save the basename to avoid path issues (too long for display,
@@ -2858,8 +2858,8 @@ allocate_compunit_symtab (struct objfile *objfile, const char *name)
void
add_compunit_symtab_to_objfile (struct compunit_symtab *cu)
{
- cu->next = cu->objfile->compunit_symtabs;
- cu->objfile->compunit_symtabs = cu;
+ cu->next = cu->objfile ()->compunit_symtabs;
+ cu->objfile ()->compunit_symtabs = cu;
}
diff --git a/gdb/symmisc.c b/gdb/symmisc.c
index cd51f6645c56..19c387d852d7 100644
--- a/gdb/symmisc.c
+++ b/gdb/symmisc.c
@@ -957,7 +957,7 @@ maintenance_print_one_line_table (struct symtab *symtab, void *data)
struct linetable *linetable;
struct objfile *objfile;
- objfile = symtab->compunit_symtab->objfile;
+ objfile = symtab->compunit_symtab->objfile ();
printf_filtered (_("objfile: %ps ((struct objfile *) %s)\n"),
styled_string (file_name_style.style (),
objfile_name (objfile)),
diff --git a/gdb/symtab.c b/gdb/symtab.c
index a32e561ed774..44b5bfb43bbf 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -338,7 +338,7 @@ compunit_symtab::find_call_site (CORE_ADDR pc) const
return nullptr;
CORE_ADDR delta
- = this->objfile->section_offsets[COMPUNIT_BLOCK_LINE_SECTION (this)];
+ = this->objfile ()->section_offsets[COMPUNIT_BLOCK_LINE_SECTION (this)];
CORE_ADDR unrelocated_pc = pc - delta;
struct call_site call_site_local (unrelocated_pc, nullptr, nullptr);
diff --git a/gdb/symtab.h b/gdb/symtab.h
index 55d1b772c457..995026bbf82f 100644
--- a/gdb/symtab.h
+++ b/gdb/symtab.h
@@ -1449,6 +1449,16 @@ struct symtab
struct compunit_symtab
{
+ struct objfile *objfile () const
+ {
+ return m_objfile;
+ }
+
+ void set_objfile (struct objfile *objfile)
+ {
+ m_objfile = objfile;
+ }
+
/* Set m_call_site_htab. */
void set_call_site_htab (htab_t call_site_htab);
@@ -1459,7 +1469,7 @@ struct compunit_symtab
struct compunit_symtab *next;
/* Object file from which this symtab information was read. */
- struct objfile *objfile;
+ struct objfile *m_objfile;
/* Name of the symtab.
This is *not* intended to be a usable filename, and is
@@ -1535,7 +1545,7 @@ struct compunit_symtab
using compunit_symtab_range = next_range<compunit_symtab>;
-#define COMPUNIT_OBJFILE(cust) ((cust)->objfile)
+#define COMPUNIT_OBJFILE(cust) ((cust)->objfile ())
#define COMPUNIT_FILETABS(cust) ((cust)->filetabs)
#define COMPUNIT_DEBUGFORMAT(cust) ((cust)->debugformat)
#define COMPUNIT_PRODUCER(cust) ((cust)->producer)
--
2.34.1
More information about the Gdb-patches
mailing list