diff mbox series

[v2,04/13] gdb: move sect_offset and cu_offset to dwarf2/types.h

Message ID 20221214033441.499512-5-simon.marchi@polymtl.ca
State Committed
Headers
Series Make frame_info_ptr automatic |

Commit Message

Simon Marchi Dec. 14, 2022, 3:34 a.m. UTC 
  From: Simon Marchi <simon.marchi@efficios.com>

I want to move the call_site stuff out of gdbtypes.h, to a new header
file, to break some cyclic include problem.  The call_site stuff uses
cu_offset, also defined in gdbtypes.h, so cu_offset also needs to move
somewhere else (otherwise, call-site.h will need to include gdbtypes.h,
and we are back to square 1).  I could move cu_offset to the future new
file dwarf2/call-site.h, but it doesn't sound like a good place for it,
at cu_offset is not specific to call sites, it's used throughout
dwarf2/.  So, move it to its own file, dwarf2/types.h.  For now,
gdbtypes.h includes dwarf2/types.h, but that will be removed once the
call site stuff is moved to its own file.

Move sect_offset with it too.  sect_offset is not a DWARF-specific
concept, but for the moment it is only used in dwarf2/.

Change-Id: I1fd2a3b7b67dee789c4874244b044bde7db43d8e
---
 gdb/dwarf2/types.h | 39 +++++++++++++++++++++++++++++++++++++++
 gdb/gdbtypes.h     | 18 +-----------------
 2 files changed, 40 insertions(+), 17 deletions(-)
 create mode 100644 gdb/dwarf2/types.h
diff mbox series

Patch

diff --git a/gdb/dwarf2/types.h b/gdb/dwarf2/types.h
new file mode 100644
index 000000000000..463dc485925f
--- /dev/null
+++ b/gdb/dwarf2/types.h
@@ -0,0 +1,39 @@ 
+/* Common internal types for the DWARF reader
+
+   Copyright (C) 2017-2022 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+#ifndef DWARF2_TYPES_H
+#define DWARF2_TYPES_H
+
+#include "gdbsupport/offset-type.h"
+
+/* Offset relative to the start of its containing CU (compilation
+   unit).  */
+DEFINE_OFFSET_TYPE (cu_offset, unsigned int);
+
+/* Offset relative to the start of its .debug_info or .debug_types
+   section.  */
+DEFINE_OFFSET_TYPE (sect_offset, uint64_t);
+
+static inline char *
+sect_offset_str (sect_offset offset)
+{
+  return hex_string (to_underlying (offset));
+}
+
+#endif /* DWARF2_TYPES_H */
diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
index 8fc5c97c95aa..fbe46323216f 100644
--- a/gdb/gdbtypes.h
+++ b/gdb/gdbtypes.h
@@ -57,6 +57,7 @@ 
 #include "gdbsupport/gdb_obstack.h"
 #include "gmp-utils.h"
 #include "frame-info.h"
+#include "dwarf2/types.h"
 
 /* Forward declarations for prototypes.  */
 struct field;
@@ -66,23 +67,6 @@  struct language_defn;
 struct dwarf2_per_cu_data;
 struct dwarf2_per_objfile;
 
-/* These declarations are DWARF-specific as some of the gdbtypes.h data types
-   are already DWARF-specific.  */
-
-/* * Offset relative to the start of its containing CU (compilation
-   unit).  */
-DEFINE_OFFSET_TYPE (cu_offset, unsigned int);
-
-/* * Offset relative to the start of its .debug_info or .debug_types
-   section.  */
-DEFINE_OFFSET_TYPE (sect_offset, uint64_t);
-
-static inline char *
-sect_offset_str (sect_offset offset)
-{
-  return hex_string (to_underlying (offset));
-}
-
 /* Some macros for char-based bitfields.  */
 
 #define B_SET(a,x)	((a)[(x)>>3] |= (1 << ((x)&7)))