From: Simon Marchi <simon.marchi@polymtl.ca>
To: gdb-patches@sourceware.org
Cc: Simon Marchi <simon.marchi@efficios.com>
Subject: [PATCH v2 02/13] gdb: move compile_instance to compile/compile.h
Date: Tue, 13 Dec 2022 22:34:30 -0500 [thread overview]
Message-ID: <20221214033441.499512-3-simon.marchi@polymtl.ca> (raw)
In-Reply-To: <20221214033441.499512-1-simon.marchi@polymtl.ca>
From: Simon Marchi <simon.marchi@efficios.com>
struct compile_instance needs to be visible to users, since we use
std::unique<compile_instance>. language.c and c-lang.c currently
includes compile-internal.h for this reason, which kind of defeats the
purpose of having an "internal" header file.
Change-Id: Iedffe5f1173b3de7bdc1be533ee2a68e6f6c549f
---
gdb/c-lang.c | 1 -
gdb/c-lang.h | 1 +
gdb/compile/compile-c.h | 1 +
gdb/compile/compile-cplus.h | 1 +
gdb/compile/compile-internal.h | 116 ---------------------------------
gdb/compile/compile.h | 116 +++++++++++++++++++++++++++++++++
gdb/language.c | 1 -
7 files changed, 119 insertions(+), 118 deletions(-)
diff --git a/gdb/c-lang.c b/gdb/c-lang.c
index 46c0da0ff797..0a10619f38fe 100644
--- a/gdb/c-lang.c
+++ b/gdb/c-lang.c
@@ -36,7 +36,6 @@
#include <ctype.h>
#include "gdbcore.h"
#include "gdbarch.h"
-#include "compile/compile-internal.h"
#include "c-exp.h"
/* Given a C string type, STR_TYPE, return the corresponding target
diff --git a/gdb/c-lang.h b/gdb/c-lang.h
index 652f147f6569..c2db16e0bf07 100644
--- a/gdb/c-lang.h
+++ b/gdb/c-lang.h
@@ -26,6 +26,7 @@ struct language_arch_info;
struct type_print_options;
struct parser_state;
+#include "compile/compile.h"
#include "value.h"
#include "macroexp.h"
#include "gdbsupport/enum-flags.h"
diff --git a/gdb/compile/compile-c.h b/gdb/compile/compile-c.h
index 4f7e5205eb7e..db17d5eedeb6 100644
--- a/gdb/compile/compile-c.h
+++ b/gdb/compile/compile-c.h
@@ -17,6 +17,7 @@
#ifndef COMPILE_COMPILE_C_H
#define COMPILE_COMPILE_C_H
+#include "compile/compile.h"
#include "gdbsupport/enum-flags.h"
#include "gcc-c-plugin.h"
diff --git a/gdb/compile/compile-cplus.h b/gdb/compile/compile-cplus.h
index 4f633e3eb585..b17f1c9c40a6 100644
--- a/gdb/compile/compile-cplus.h
+++ b/gdb/compile/compile-cplus.h
@@ -17,6 +17,7 @@
#ifndef COMPILE_COMPILE_CPLUS_H
#define COMPILE_COMPILE_CPLUS_H
+#include "compile/compile.h"
#include "gdbsupport/enum-flags.h"
#include "gcc-cp-plugin.h"
#include "symtab.h"
diff --git a/gdb/compile/compile-internal.h b/gdb/compile/compile-internal.h
index 0d3fcd76f0c0..4607f33738cc 100644
--- a/gdb/compile/compile-internal.h
+++ b/gdb/compile/compile-internal.h
@@ -24,122 +24,6 @@
extern bool compile_debug;
-struct block;
-
-/* An object of this type holds state associated with a given
- compilation job. */
-
-class compile_instance
-{
-public:
- compile_instance (struct gcc_base_context *gcc_fe, const char *options);
-
- virtual ~compile_instance ()
- {
- m_gcc_fe->ops->destroy (m_gcc_fe);
- }
-
- /* Returns the GCC options to be passed during compilation. */
- const std::string &gcc_target_options () const
- {
- return m_gcc_target_options;
- }
-
- /* Query the type cache for TYPE, returning the compiler's
- type for it in RET. */
- bool get_cached_type (struct type *type, gcc_type *ret) const;
-
- /* Insert GCC_TYPE into the type cache for TYPE.
-
- It is ok for a given type to be inserted more than once, provided that
- the exact same association is made each time. */
- void insert_type (struct type *type, gcc_type gcc_type);
-
- /* Associate SYMBOL with some error text. */
- void insert_symbol_error (const struct symbol *sym, const char *text);
-
- /* Emit the error message corresponding to SYM, if one exists, and
- arrange for it not to be emitted again. */
- void error_symbol_once (const struct symbol *sym);
-
- /* These currently just forward to the underlying ops
- vtable. */
-
- /* Set the plug-in print callback. */
- void set_print_callback (void (*print_function) (void *, const char *),
- void *datum);
-
- /* Return the plug-in's front-end version. */
- unsigned int version () const;
-
- /* Set the plug-in's verbosity level. Nop for GCC_FE_VERSION_0. */
- void set_verbose (int level);
-
- /* Set the plug-in driver program. Nop for GCC_FE_VERSION_0. */
- void set_driver_filename (const char *filename);
-
- /* Set the regular expression used to match the configury triplet
- prefix to the compiler. Nop for GCC_FE_VERSION_0. */
- void set_triplet_regexp (const char *regexp);
-
- /* Set compilation arguments. REGEXP is only used for protocol
- version GCC_FE_VERSION_0. */
- gdb::unique_xmalloc_ptr<char> set_arguments (int argc, char **argv,
- const char *regexp = NULL);
-
- /* Set the filename of the program to compile. Nop for GCC_FE_VERSION_0. */
- void set_source_file (const char *filename);
-
- /* Compile the previously specified source file to FILENAME.
- VERBOSE_LEVEL is only used for protocol version GCC_FE_VERSION_0. */
- bool compile (const char *filename, int verbose_level = -1);
-
- /* Set the scope type for this compile. */
- void set_scope (enum compile_i_scope_types scope)
- {
- m_scope = scope;
- }
-
- /* Return the scope type. */
- enum compile_i_scope_types scope () const
- {
- return m_scope;
- }
-
- /* Set the block to be used for symbol searches. */
- void set_block (const struct block *block)
- {
- m_block = block;
- }
-
- /* Return the search block. */
- const struct block *block () const
- {
- return m_block;
- }
-
-protected:
-
- /* The GCC front end. */
- struct gcc_base_context *m_gcc_fe;
-
- /* The "scope" of this compilation. */
- enum compile_i_scope_types m_scope;
-
- /* The block in which an expression is being parsed. */
- const struct block *m_block;
-
- /* Specify "-std=gnu11", "-std=gnu++11" or similar. These options are put
- after CU's DW_AT_producer compilation options to override them. */
- std::string m_gcc_target_options;
-
- /* Map from gdb types to gcc types. */
- htab_up m_type_map;
-
- /* Map from gdb symbols to gcc error messages to emit. */
- htab_up m_symbol_err_map;
-};
-
/* Define header and footers for different scopes. */
/* A simple scope just declares a function named "_gdb_expr", takes no
diff --git a/gdb/compile/compile.h b/gdb/compile/compile.h
index d517660dd40a..877509544787 100644
--- a/gdb/compile/compile.h
+++ b/gdb/compile/compile.h
@@ -18,6 +18,8 @@
#ifndef COMPILE_COMPILE_H
#define COMPILE_COMPILE_H
+#include "gcc-c-interface.h"
+
struct ui_file;
struct gdbarch;
struct dwarf2_per_cu_data;
@@ -25,6 +27,120 @@ struct dwarf2_per_objfile;
struct symbol;
struct dynamic_prop;
+/* An object of this type holds state associated with a given
+ compilation job. */
+
+class compile_instance
+{
+public:
+ compile_instance (struct gcc_base_context *gcc_fe, const char *options);
+
+ virtual ~compile_instance ()
+ {
+ m_gcc_fe->ops->destroy (m_gcc_fe);
+ }
+
+ /* Returns the GCC options to be passed during compilation. */
+ const std::string &gcc_target_options () const
+ {
+ return m_gcc_target_options;
+ }
+
+ /* Query the type cache for TYPE, returning the compiler's
+ type for it in RET. */
+ bool get_cached_type (struct type *type, gcc_type *ret) const;
+
+ /* Insert GCC_TYPE into the type cache for TYPE.
+
+ It is ok for a given type to be inserted more than once, provided that
+ the exact same association is made each time. */
+ void insert_type (struct type *type, gcc_type gcc_type);
+
+ /* Associate SYMBOL with some error text. */
+ void insert_symbol_error (const struct symbol *sym, const char *text);
+
+ /* Emit the error message corresponding to SYM, if one exists, and
+ arrange for it not to be emitted again. */
+ void error_symbol_once (const struct symbol *sym);
+
+ /* These currently just forward to the underlying ops
+ vtable. */
+
+ /* Set the plug-in print callback. */
+ void set_print_callback (void (*print_function) (void *, const char *),
+ void *datum);
+
+ /* Return the plug-in's front-end version. */
+ unsigned int version () const;
+
+ /* Set the plug-in's verbosity level. Nop for GCC_FE_VERSION_0. */
+ void set_verbose (int level);
+
+ /* Set the plug-in driver program. Nop for GCC_FE_VERSION_0. */
+ void set_driver_filename (const char *filename);
+
+ /* Set the regular expression used to match the configury triplet
+ prefix to the compiler. Nop for GCC_FE_VERSION_0. */
+ void set_triplet_regexp (const char *regexp);
+
+ /* Set compilation arguments. REGEXP is only used for protocol
+ version GCC_FE_VERSION_0. */
+ gdb::unique_xmalloc_ptr<char> set_arguments (int argc, char **argv,
+ const char *regexp = NULL);
+
+ /* Set the filename of the program to compile. Nop for GCC_FE_VERSION_0. */
+ void set_source_file (const char *filename);
+
+ /* Compile the previously specified source file to FILENAME.
+ VERBOSE_LEVEL is only used for protocol version GCC_FE_VERSION_0. */
+ bool compile (const char *filename, int verbose_level = -1);
+
+ /* Set the scope type for this compile. */
+ void set_scope (enum compile_i_scope_types scope)
+ {
+ m_scope = scope;
+ }
+
+ /* Return the scope type. */
+ enum compile_i_scope_types scope () const
+ {
+ return m_scope;
+ }
+
+ /* Set the block to be used for symbol searches. */
+ void set_block (const struct block *block)
+ {
+ m_block = block;
+ }
+
+ /* Return the search block. */
+ const struct block *block () const
+ {
+ return m_block;
+ }
+
+protected:
+
+ /* The GCC front end. */
+ struct gcc_base_context *m_gcc_fe;
+
+ /* The "scope" of this compilation. */
+ enum compile_i_scope_types m_scope;
+
+ /* The block in which an expression is being parsed. */
+ const struct block *m_block;
+
+ /* Specify "-std=gnu11", "-std=gnu++11" or similar. These options are put
+ after CU's DW_AT_producer compilation options to override them. */
+ std::string m_gcc_target_options;
+
+ /* Map from gdb types to gcc types. */
+ htab_up m_type_map;
+
+ /* Map from gdb symbols to gcc error messages to emit. */
+ htab_up m_symbol_err_map;
+};
+
/* Public function that is called from compile_control case in the
expression command. GDB returns either a CMD, or a CMD_STRING, but
never both. */
diff --git a/gdb/language.c b/gdb/language.c
index 3962ee8fa246..df4a06cbcc2a 100644
--- a/gdb/language.c
+++ b/gdb/language.c
@@ -46,7 +46,6 @@
#include "c-lang.h"
#include <algorithm>
#include "gdbarch.h"
-#include "compile/compile-internal.h"
static void set_range_case (void);
--
2.38.1
next prev parent reply other threads:[~2022-12-14 3:34 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-14 3:34 [PATCH v2 00/13] Make frame_info_ptr automatic Simon Marchi
2022-12-14 3:34 ` [PATCH v2 01/13] gdb: move type_map_instance to compile/compile.c Simon Marchi
2022-12-14 3:34 ` Simon Marchi [this message]
2022-12-14 3:34 ` [PATCH v2 03/13] gdb: remove language.h include from frame.h Simon Marchi
2022-12-14 3:34 ` [PATCH v2 04/13] gdb: move sect_offset and cu_offset to dwarf2/types.h Simon Marchi
2022-12-14 3:34 ` [PATCH v2 05/13] gdb: move call site types to call-site.h Simon Marchi
2022-12-14 3:34 ` [PATCH v2 06/13] gdb: move frame_info_ptr to frame.{c,h} Simon Marchi
2022-12-20 17:01 ` Bruno Larsen
2023-01-03 18:59 ` Simon Marchi
2022-12-14 3:34 ` [PATCH v2 07/13] gdb: add frame_id::user_created_p Simon Marchi
2022-12-14 3:34 ` [PATCH v2 08/13] gdb: add user-created frames to stash Simon Marchi
2022-12-14 3:34 ` [PATCH v2 09/13] gdb: add create_new_frame(frame_id) overload Simon Marchi
2022-12-14 3:34 ` [PATCH v2 10/13] gdb: make it possible to restore selected user-created frames Simon Marchi
2022-12-14 3:34 ` [PATCH v2 11/13] gdb: make user-created frames reinflatable Simon Marchi
2023-01-23 12:57 ` Tom de Vries
2023-01-23 14:34 ` Luis Machado
2023-01-24 3:55 ` Simon Marchi
2023-01-24 8:22 ` Luis Machado
2023-01-25 3:45 ` Simon Marchi
2023-01-30 8:49 ` Luis Machado
2023-01-30 16:20 ` Simon Marchi
2022-12-14 3:34 ` [PATCH v2 12/13] gdb: make frame_info_ptr grab frame level and id on construction Simon Marchi
2022-12-14 3:34 ` [PATCH v2 13/13] gdb: make frame_info_ptr auto-reinflatable Simon Marchi
2022-12-20 16:57 ` [PATCH v2 00/13] Make frame_info_ptr automatic Bruno Larsen
2023-01-03 19:00 ` Simon Marchi
2023-01-03 19:09 ` Simon Marchi
2023-01-18 18:10 ` Tom Tromey
2023-01-19 3:40 ` Simon Marchi
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=20221214033441.499512-3-simon.marchi@polymtl.ca \
--to=simon.marchi@polymtl.ca \
--cc=gdb-patches@sourceware.org \
--cc=simon.marchi@efficios.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).