Use gdbarch for probe::get_argument_count
authorAlan Hayward <alan.hayward@arm.com>
Wed, 21 Aug 2019 14:24:02 +0000 (15:24 +0100)
committerAlan Hayward <alan.hayward@arm.com>
Mon, 2 Sep 2019 12:58:48 +0000 (13:58 +0100)
The probe function get_argument_count does not need a frame, only
the current gdbarch.  Switch the code to pass gdbarch instead.
No functional changes.

gdb/ChangeLog:

* break-catch-throw.c (fetch_probe_arguments): Use gdbarch.
* dtrace-probe.c (dtrace_probe::get_argument_count): Likewise.
* probe.c (probe_safe_evaluate_at_pc) (compute_probe_arg)
(compile_probe_arg): Likewise.
* probe.h (get_argument_count): Likewise.
* solib-svr4.c (solib_event_probe_action): Likewise.
* stap-probe.c (stap_probe::get_argument_count): Likewise.

gdb/ChangeLog
gdb/break-catch-throw.c
gdb/dtrace-probe.c
gdb/probe.c
gdb/probe.h
gdb/solib-svr4.c
gdb/stap-probe.c

index e35ff66c54b46a732f517cff84bfd20d5e6d6590..d0e4258c4f7313068b8167580ae1389a55a9d39d 100644 (file)
@@ -1,3 +1,13 @@
+2019-09-02  Alan Hayward  <alan.hayward@arm.com>
+
+       * break-catch-throw.c (fetch_probe_arguments): Use gdbarch.
+       * dtrace-probe.c (dtrace_probe::get_argument_count): Likewise.
+       * probe.c (probe_safe_evaluate_at_pc) (compute_probe_arg)
+       (compile_probe_arg): Likewise.
+       * probe.h (get_argument_count): Likewise.
+       * solib-svr4.c (solib_event_probe_action): Likewise.
+       * stap-probe.c (stap_probe::get_argument_count): Likewise.
+
 2019-09-02  Alan Hayward  <alan.hayward@arm.com>
 
        * solib-svr4.c (svr4_find_and_create_probe_breakpoints): Move
index 2d91285312bb7e51d0b3aa1c449ea08ced2e6a92..e9523a6eb86014a207b74ffbfa137ae6799467d3 100644 (file)
@@ -115,7 +115,7 @@ fetch_probe_arguments (struct value **arg0, struct value **arg1)
          && pc_probe.prob->get_name () != "rethrow"))
     error (_("not stopped at a C++ exception catchpoint"));
 
-  n_args = pc_probe.prob->get_argument_count (frame);
+  n_args = pc_probe.prob->get_argument_count (get_frame_arch (frame));
   if (n_args < 2)
     error (_("C++ exception catchpoint has too few arguments"));
 
index 2d92edb11cd37ffab01c6e45d2b4e5c32ca611d2..e9e71fd4c9e82036dbf9464ed9393ea5580ca37c 100644 (file)
@@ -122,7 +122,7 @@ public:
   CORE_ADDR get_relocated_address (struct objfile *objfile) override;
 
   /* See probe.h.  */
-  unsigned get_argument_count (struct frame_info *frame) override;
+  unsigned get_argument_count (struct gdbarch *gdbarch) override;
 
   /* See probe.h.  */
   bool can_evaluate_arguments () const override;
@@ -693,7 +693,7 @@ dtrace_probe::get_relocated_address (struct objfile *objfile)
 /* Implementation of the get_argument_count method.  */
 
 unsigned
-dtrace_probe::get_argument_count (struct frame_info *frame)
+dtrace_probe::get_argument_count (struct gdbarch *gdbarch)
 {
   return m_args.size ();
 }
index cdc6e021d2e7682f23e1133c41a7bab97f6e249f..8b108d6b02dce8e8dc87a7a01244199b740e93e6 100644 (file)
@@ -695,7 +695,7 @@ probe_safe_evaluate_at_pc (struct frame_info *frame, unsigned n)
   if (!probe.prob)
     return NULL;
 
-  n_args = probe.prob->get_argument_count (frame);
+  n_args = probe.prob->get_argument_count (get_frame_arch (frame));
   if (n >= n_args)
     return NULL;
 
@@ -818,7 +818,7 @@ compute_probe_arg (struct gdbarch *arch, struct internalvar *ivar,
   if (pc_probe.prob == NULL)
     error (_("No probe at PC %s"), core_addr_to_string (pc));
 
-  n_args = pc_probe.prob->get_argument_count (frame);
+  n_args = pc_probe.prob->get_argument_count (arch);
   if (sel == -1)
     return value_from_longest (builtin_type (arch)->builtin_int, n_args);
 
@@ -840,7 +840,6 @@ compile_probe_arg (struct internalvar *ivar, struct agent_expr *expr,
   int sel = (int) (uintptr_t) data;
   struct bound_probe pc_probe;
   int n_args;
-  struct frame_info *frame = get_selected_frame (NULL);
 
   /* SEL == -1 means "_probe_argc".  */
   gdb_assert (sel >= -1);
@@ -849,7 +848,7 @@ compile_probe_arg (struct internalvar *ivar, struct agent_expr *expr,
   if (pc_probe.prob == NULL)
     error (_("No probe at PC %s"), core_addr_to_string (pc));
 
-  n_args = pc_probe.prob->get_argument_count (frame);
+  n_args = pc_probe.prob->get_argument_count (expr->gdbarch);
 
   if (sel == -1)
     {
index 8abf69e3540ec983f1524b012a57c8ca03fc4934..7410c5aadf69329c6af4cded5ee952f93a04d989 100644 (file)
@@ -131,7 +131,7 @@ public:
 
   /* Return the number of arguments of the probe.  This function can
      throw an exception.  */
-  virtual unsigned get_argument_count (struct frame_info *frame) = 0;
+  virtual unsigned get_argument_count (struct gdbarch *gdbarch) = 0;
 
   /* Return 1 if the probe interface can evaluate the arguments of
      probe, zero otherwise.  See the comments on
index b21eacb68ffb75cc30aa421dc38d115c4dcee34a..2a44dd69858ae7b0ade6cb07f47f1c1f3ccef3b9 100644 (file)
@@ -1756,7 +1756,7 @@ solib_event_probe_action (struct probe_and_action *pa)
        arg2: struct link_map *new (optional, for incremental updates)  */
   try
     {
-      probe_argc = pa->prob->get_argument_count (frame);
+      probe_argc = pa->prob->get_argument_count (get_frame_arch (frame));
     }
   catch (const gdb_exception_error &ex)
     {
index b6de873d2c935a42e21c6b19bcb667a3b5959348..700b657967d3d8471677881b7a278ef4ee2ac26f 100644 (file)
@@ -136,7 +136,7 @@ public:
   CORE_ADDR get_relocated_address (struct objfile *objfile) override;
 
   /* See probe.h.  */
-  unsigned get_argument_count (struct frame_info *frame) override;
+  unsigned get_argument_count (struct gdbarch *gdbarch) override;
 
   /* See probe.h.  */
   bool can_evaluate_arguments () const override;
@@ -1301,10 +1301,8 @@ stap_probe::get_relocated_address (struct objfile *objfile)
    argument string.  */
 
 unsigned
-stap_probe::get_argument_count (struct frame_info *frame)
+stap_probe::get_argument_count (struct gdbarch *gdbarch)
 {
-  struct gdbarch *gdbarch = get_frame_arch (frame);
-
   if (!m_have_parsed_args)
     {
       if (this->can_evaluate_arguments ())