public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
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


  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).