OSDN Git Service

Cleanup free_agent_expr cleanups.
authorcagney <cagney>
Mon, 15 May 2000 06:15:27 +0000 (06:15 +0000)
committercagney <cagney>
Mon, 15 May 2000 06:15:27 +0000 (06:15 +0000)
gdb/ChangeLog
gdb/ax-gdb.c
gdb/ax-general.c
gdb/ax.h
gdb/tracepoint.c

index 04b6f79..5e655b1 100644 (file)
@@ -1,3 +1,13 @@
+Mon May 15 16:05:46 2000  Andrew Cagney  <cagney@b1.cygnus.com>
+
+       * ax.h (make_cleanup_free_agent_expr): Add declaration.
+       * ax-general.c (do_free_agent_expr_cleanup): 
+       (make_cleanup_free_agent_expr): New functions.
+
+       * tracepoint.c (validate_actionline, encode_actions), ax-gdb.c
+       (expr_to_agent, gen_trace_for_expr, agent_command): Replace calls
+       to make_cleanup.
+
 Mon May 15 14:06:23 2000  Andrew Cagney  <cagney@b1.cygnus.com>
 
        * breakpoint.h (make_cleanup_delete_breakpoint,
index 291ac75..08ce732 100644 (file)
@@ -1798,7 +1798,7 @@ expr_to_agent (expr, value)
   struct agent_expr *ax = new_agent_expr (0);
   union exp_element *pc;
 
-  old_chain = make_cleanup ((make_cleanup_func) free_agent_expr, ax);
+  old_chain = make_cleanup_free_agent_expr (ax);
 
   pc = expr->elts;
   trace_kludge = 0;
@@ -1854,7 +1854,7 @@ gen_trace_for_expr (scope, expr)
   union exp_element *pc;
   struct axs_value value;
 
-  old_chain = make_cleanup ((make_cleanup_func) free_agent_expr, ax);
+  old_chain = make_cleanup_free_agent_expr (ax);
 
   pc = expr->elts;
   trace_kludge = 1;
@@ -1925,7 +1925,7 @@ agent_command (exp, from_tty)
   expr = parse_expression (exp);
   old_chain = make_cleanup (free_current_contents, &expr);
   agent = gen_trace_for_expr (fi->pc, expr);
-  make_cleanup ((make_cleanup_func) free_agent_expr, agent);
+  make_cleanup_free_agent_expr (agent);
   ax_print (gdb_stdout, agent);
 
   /* It would be nice to call ax_reqs here to gather some general info
index 76e9bb4..4612e8e 100644 (file)
@@ -1,23 +1,22 @@
 /* Functions for manipulating expressions designed to be executed on the agent
-   Copyright 1998 Free Software Foundation, Inc.
+   Copyright 1998, 2000 Free Software Foundation, Inc.
 
-This file is part of GDB.
+   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 2 of the License, or
-(at your option) any later version.
+   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 2 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.
+   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, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
-
-/* $Id$ */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 /* Despite what the above comment says about this file being part of
    GDB, we would like to keep these functions free of GDB
@@ -27,6 +26,15 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "defs.h"
 #include "ax.h"
 
+#include "value.h"
+
+static void grow_expr PARAMS ((struct agent_expr * x, int n));
+
+static void append_const PARAMS ((struct agent_expr * x, LONGEST val, int n));
+
+static LONGEST read_const PARAMS ((struct agent_expr * x, int o, int n));
+
+static void generic_ext PARAMS ((struct agent_expr * x, enum agent_op op, int n));
 \f
 /* Functions for building expressions.  */
 
@@ -36,10 +44,10 @@ new_agent_expr (scope)
      CORE_ADDR scope;
 {
   struct agent_expr *x = xmalloc (sizeof (*x));
-  x->len  = 0;
+  x->len = 0;
   x->size = 1;                 /* Change this to a larger value once
                                   reallocation code is tested.  */
-  x->buf   = xmalloc (x->size);
+  x->buf = xmalloc (x->size);
   x->scope = scope;
 
   return x;
@@ -54,6 +62,18 @@ free_agent_expr (x)
   free (x);
 }
 
+static void
+do_free_agent_expr_cleanup (void *x)
+{
+  free_agent_expr (x);
+}
+
+struct cleanup *
+make_cleanup_free_agent_expr (struct agent_expr *x)
+{
+  return make_cleanup (do_free_agent_expr_cleanup, x);
+}
+
 
 /* Make sure that X has room for at least N more bytes.  This doesn't
    affect the length, just the allocated size.  */
@@ -108,7 +128,7 @@ read_const (x, o, n)
 
   for (i = 0; i < n; i++)
     accum = (accum << 8) | x->buf[o + i];
-  
+
   return accum;
 }
 
@@ -187,7 +207,8 @@ ax_trace_quick (x, n)
    for the target, and return the offset in EXPR of that space, so we
    can backpatch it once we do know the target offset.  Use ax_label
    to do the backpatching.  */
-int ax_goto (x, op)
+int
+ax_goto (x, op)
      struct agent_expr *x;
      enum agent_op op;
 {
@@ -201,7 +222,7 @@ int ax_goto (x, op)
 
 /* Suppose a given call to ax_goto returns some value PATCH.  When you
    know the offset TARGET that goto should jump to, call
-       ax_label (EXPR, PATCH, TARGET)
+   ax_label (EXPR, PATCH, TARGET)
    to patch TARGET into the ax_goto instruction.  */
 void
 ax_label (x, patch, target)
@@ -213,7 +234,7 @@ ax_label (x, patch, target)
      offset; that's our magic sentinel value for unpatched branches.  */
   if (target < 0 || target >= 0xffff)
     error ("GDB bug: ax-general.c (ax_label): label target out of range");
-  
+
   x->buf[patch] = (target >> 8) & 0xff;
   x->buf[patch + 1] = target & 0xff;
 }
@@ -226,7 +247,8 @@ ax_const_l (x, l)
      LONGEST l;
 {
   static enum agent_op ops[]
-    = { aop_const8, aop_const16, aop_const32, aop_const64 };
+  =
+  {aop_const8, aop_const16, aop_const32, aop_const64};
   int size;
   int op;
 
@@ -263,7 +285,8 @@ ax_const_d (x, d)
 
 /* Assemble code to push the value of register number REG on the
    stack.  */
-void ax_reg (x, reg)
+void
+ax_reg (x, reg)
      struct agent_expr *x;
      int reg;
 {
@@ -271,74 +294,75 @@ void ax_reg (x, reg)
   if (reg < 0 || reg > 0xffff)
     error ("GDB bug: ax-general.c (ax_reg): register number out of range");
   grow_expr (x, 3);
-  x->buf[x->len    ] = aop_reg;
+  x->buf[x->len] = aop_reg;
   x->buf[x->len + 1] = (reg >> 8) & 0xff;
-  x->buf[x->len + 2] = (reg     ) & 0xff;
+  x->buf[x->len + 2] = (reg) & 0xff;
   x->len += 3;
 }
+\f
 
 
-\f
 /* Functions for disassembling agent expressions, and otherwise
    debugging the expression compiler.  */
 
-struct aop_map aop_map[] = {
-  { 0, 0, 0, 0, 0 },
-  { "float", 0, 0, 0, 0 },                /* 0x01 */
-  { "add", 0, 0, 2, 1 },                  /* 0x02 */
-  { "sub", 0, 0, 2, 1 },                  /* 0x03 */
-  { "mul", 0, 0, 2, 1 },                  /* 0x04 */
-  { "div_signed", 0, 0, 2, 1 },                   /* 0x05 */
-  { "div_unsigned", 0, 0, 2, 1 },         /* 0x06 */
-  { "rem_signed", 0, 0, 2, 1 },                   /* 0x07 */
-  { "rem_unsigned", 0, 0, 2, 1 },         /* 0x08 */
-  { "lsh", 0, 0, 2, 1 },                  /* 0x09 */
-  { "rsh_signed", 0, 0, 2, 1 },                   /* 0x0a */
-  { "rsh_unsigned", 0, 0, 2, 1 },         /* 0x0b */
-  { "trace", 0, 0, 2, 0 },                /* 0x0c */
-  { "trace_quick", 1, 0, 1, 1 },          /* 0x0d */
-  { "log_not", 0, 0, 1, 1 },              /* 0x0e */
-  { "bit_and", 0, 0, 2, 1 },              /* 0x0f */
-  { "bit_or", 0, 0, 2, 1 },               /* 0x10 */
-  { "bit_xor", 0, 0, 2, 1 },              /* 0x11 */
-  { "bit_not", 0, 0, 1, 1 },              /* 0x12 */
-  { "equal", 0, 0, 2, 1 },                /* 0x13 */
-  { "less_signed", 0, 0, 2, 1 },          /* 0x14 */
-  { "less_unsigned", 0, 0, 2, 1 },        /* 0x15 */
-  { "ext", 1, 0, 1, 1 },                  /* 0x16 */
-  { "ref8", 0, 8, 1, 1 },                 /* 0x17 */
-  { "ref16", 0, 16, 1, 1 },               /* 0x18 */
-  { "ref32", 0, 32, 1, 1 },               /* 0x19 */
-  { "ref64", 0, 64, 1, 1 },               /* 0x1a */
-  { "ref_float", 0, 0, 1, 1 },            /* 0x1b */
-  { "ref_double", 0, 0, 1, 1 },                   /* 0x1c */
-  { "ref_long_double", 0, 0, 1, 1 },      /* 0x1d */
-  { "l_to_d", 0, 0, 1, 1 },               /* 0x1e */
-  { "d_to_l", 0, 0, 1, 1 },               /* 0x1f */
-  { "if_goto", 2, 0, 1, 0 },              /* 0x20 */
-  { "goto", 2, 0, 0, 0 },                 /* 0x21 */
-  { "const8", 1, 8, 0, 1 },               /* 0x22 */
-  { "const16", 2, 16, 0, 1 },             /* 0x23 */
-  { "const32", 4, 32, 0, 1 },             /* 0x24 */
-  { "const64", 8, 64, 0, 1 },             /* 0x25 */
-  { "reg", 2, 0, 0, 1 },                  /* 0x26 */
-  { "end", 0, 0, 0, 0 },                  /* 0x27 */
-  { "dup", 0, 0, 1, 2 },                  /* 0x28 */
-  { "pop", 0, 0, 1, 0 },                  /* 0x29 */
-  { "zero_ext", 1, 0, 1, 1 },             /* 0x2a */
-  { "swap", 0, 0, 2, 2 },                 /* 0x2b */
-  { 0, 0, 0, 0, 0 },                      /* 0x2c */
-  { 0, 0, 0, 0, 0 },                      /* 0x2d */
-  { 0, 0, 0, 0, 0 },                      /* 0x2e */
-  { 0, 0, 0, 0, 0 },                      /* 0x2f */
-  { "trace16", 2, 0, 1, 1 },              /* 0x30 */
+struct aop_map aop_map[] =
+{
+  {0, 0, 0, 0, 0},
+  {"float", 0, 0, 0, 0},       /* 0x01 */
+  {"add", 0, 0, 2, 1},         /* 0x02 */
+  {"sub", 0, 0, 2, 1},         /* 0x03 */
+  {"mul", 0, 0, 2, 1},         /* 0x04 */
+  {"div_signed", 0, 0, 2, 1},  /* 0x05 */
+  {"div_unsigned", 0, 0, 2, 1},        /* 0x06 */
+  {"rem_signed", 0, 0, 2, 1},  /* 0x07 */
+  {"rem_unsigned", 0, 0, 2, 1},        /* 0x08 */
+  {"lsh", 0, 0, 2, 1},         /* 0x09 */
+  {"rsh_signed", 0, 0, 2, 1},  /* 0x0a */
+  {"rsh_unsigned", 0, 0, 2, 1},        /* 0x0b */
+  {"trace", 0, 0, 2, 0},       /* 0x0c */
+  {"trace_quick", 1, 0, 1, 1}, /* 0x0d */
+  {"log_not", 0, 0, 1, 1},     /* 0x0e */
+  {"bit_and", 0, 0, 2, 1},     /* 0x0f */
+  {"bit_or", 0, 0, 2, 1},      /* 0x10 */
+  {"bit_xor", 0, 0, 2, 1},     /* 0x11 */
+  {"bit_not", 0, 0, 1, 1},     /* 0x12 */
+  {"equal", 0, 0, 2, 1},       /* 0x13 */
+  {"less_signed", 0, 0, 2, 1}, /* 0x14 */
+  {"less_unsigned", 0, 0, 2, 1},       /* 0x15 */
+  {"ext", 1, 0, 1, 1},         /* 0x16 */
+  {"ref8", 0, 8, 1, 1},                /* 0x17 */
+  {"ref16", 0, 16, 1, 1},      /* 0x18 */
+  {"ref32", 0, 32, 1, 1},      /* 0x19 */
+  {"ref64", 0, 64, 1, 1},      /* 0x1a */
+  {"ref_float", 0, 0, 1, 1},   /* 0x1b */
+  {"ref_double", 0, 0, 1, 1},  /* 0x1c */
+  {"ref_long_double", 0, 0, 1, 1},     /* 0x1d */
+  {"l_to_d", 0, 0, 1, 1},      /* 0x1e */
+  {"d_to_l", 0, 0, 1, 1},      /* 0x1f */
+  {"if_goto", 2, 0, 1, 0},     /* 0x20 */
+  {"goto", 2, 0, 0, 0},                /* 0x21 */
+  {"const8", 1, 8, 0, 1},      /* 0x22 */
+  {"const16", 2, 16, 0, 1},    /* 0x23 */
+  {"const32", 4, 32, 0, 1},    /* 0x24 */
+  {"const64", 8, 64, 0, 1},    /* 0x25 */
+  {"reg", 2, 0, 0, 1},         /* 0x26 */
+  {"end", 0, 0, 0, 0},         /* 0x27 */
+  {"dup", 0, 0, 1, 2},         /* 0x28 */
+  {"pop", 0, 0, 1, 0},         /* 0x29 */
+  {"zero_ext", 1, 0, 1, 1},    /* 0x2a */
+  {"swap", 0, 0, 2, 2},                /* 0x2b */
+  {0, 0, 0, 0, 0},             /* 0x2c */
+  {0, 0, 0, 0, 0},             /* 0x2d */
+  {0, 0, 0, 0, 0},             /* 0x2e */
+  {0, 0, 0, 0, 0},             /* 0x2f */
+  {"trace16", 2, 0, 1, 1},     /* 0x30 */
 };
 
 
 /* Disassemble the expression EXPR, writing to F.  */
 void
 ax_print (f, x)
-     GDB_FILE *f;
+     struct ui_file *f;
      struct agent_expr *x;
 {
   int i;
@@ -349,13 +373,13 @@ ax_print (f, x)
   if ((sizeof (aop_map) / sizeof (aop_map[0]))
       != aop_last)
     error ("GDB bug: ax-general.c (ax_print): opcode map out of sync");
-  
-  for (i = 0; i < x->len; )
+
+  for (i = 0; i < x->len;)
     {
       enum agent_op op = x->buf[i];
 
       if (op >= (sizeof (aop_map) / sizeof (aop_map[0]))
-         || ! aop_map[op].name)
+         || !aop_map[op].name)
        {
          fprintf_filtered (f, "%3d  <bad opcode %02x>\n", i, op);
          i++;
@@ -372,7 +396,7 @@ ax_print (f, x)
       if (aop_map[op].op_size > 0)
        {
          fputs_filtered (" ", f);
-         
+
          print_longest (f, 'd', 0,
                         read_const (x, i + 1, aop_map[op].op_size));
        }
@@ -433,13 +457,13 @@ ax_reqs (ax, reqs)
 
       op = &aop_map[ax->buf[i]];
 
-      if (! op->name)
+      if (!op->name)
        {
          reqs->flaw = agent_flaw_bad_instruction;
          free (reg_mask);
          return;
        }
-       
+
       if (i + 1 + op->op_size > ax->len)
        {
          reqs->flaw = agent_flaw_incomplete_instruction;
@@ -470,8 +494,8 @@ ax_reqs (ax, reqs)
        reqs->max_data_size = op->data_size;
 
       /* For jump instructions, check that the target is a valid
-        offset.  If it is, record the fact that that location is a
-        jump target, and record the height we expect there.  */
+         offset.  If it is, record the fact that that location is a
+         jump target, and record the height we expect there.  */
       if (aop_goto == op - aop_map
          || aop_if_goto == op - aop_map)
        {
@@ -498,13 +522,13 @@ ax_reqs (ax, reqs)
              heights[target] = height;
            }
        }
-      
+
       /* For unconditional jumps with a successor, check that the
          successor is a target, and pick up its stack height.  */
       if (aop_goto == op - aop_map
          && i + 3 < ax->len)
        {
-         if (! targets[i + 3])
+         if (!targets[i + 3])
            {
              reqs->flaw = agent_flaw_hole;
              free (reg_mask);
@@ -524,9 +548,9 @@ ax_reqs (ax, reqs)
          if (byte >= reg_mask_len)
            {
              /* It's not appropriate to double here.  This isn't a
-                 string buffer.  */
+                string buffer.  */
              int new_len = byte + 1;
-             reg_mask = xrealloc (reg_mask, 
+             reg_mask = xrealloc (reg_mask,
                                   new_len * sizeof (reg_mask[0]));
              memset (reg_mask + reg_mask_len, 0,
                      (new_len - reg_mask_len) * sizeof (reg_mask[0]));
index 2ee0f0b..f6860b3 100644 (file)
--- a/gdb/ax.h
+++ b/gdb/ax.h
@@ -1,23 +1,22 @@
 /* Definitions for expressions designed to be executed on the agent
-   Copyright 1998 Free Software Foundation, Inc.
+   Copyright 1998, 2000 Free Software Foundation, Inc.
 
-This file is part of GDB.
+   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 2 of the License, or
-(at your option) any later version.
+   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 2 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.
+   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, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
-
-/* $Id$ */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 #ifndef AGENTEXPR_H
 #define AGENTEXPR_H
@@ -49,8 +48,8 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
    is independent of symbolic information.  This means the agent can
    evaluate them on the fly without reference to data only available
    to the host GDB.  */
-
 \f
+
 /* Agent expression data structures.  */
 
 /* The type of an element of the agent expression stack.
@@ -58,18 +57,20 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
    the value itself has no typing information.  GDB generates all
    bytecode streams, so we don't have to worry about type errors.  */
 
-union agent_val {
-  LONGEST l;
-  DOUBLEST d;
-};
+union agent_val
+  {
+    LONGEST l;
+    DOUBLEST d;
+  };
 
 /* A buffer containing a agent expression.  */
-struct agent_expr {
-  unsigned char *buf;
-  int len;                     /* number of characters used */
-  int size;                    /* allocated size */
-  CORE_ADDR scope;
-};
+struct agent_expr
+  {
+    unsigned char *buf;
+    int len;                   /* number of characters used */
+    int size;                  /* allocated size */
+    CORE_ADDR scope;
+  };
 
 
 
@@ -84,56 +85,57 @@ struct agent_expr {
    agentexpr.h behaves like an opcode map.  If you want to see them
    grouped logically, see doc/agentexpr.texi.  */
 
-enum agent_op {
-  aop_float = 0x01,
-  aop_add = 0x02,
-  aop_sub = 0x03,
-  aop_mul = 0x04,
-  aop_div_signed = 0x05,
-  aop_div_unsigned = 0x06,
-  aop_rem_signed = 0x07,
-  aop_rem_unsigned = 0x08,
-  aop_lsh = 0x09,
-  aop_rsh_signed = 0x0a,
-  aop_rsh_unsigned = 0x0b,
-  aop_trace = 0x0c,
-  aop_trace_quick = 0x0d,
-  aop_log_not = 0x0e,
-  aop_bit_and = 0x0f,
-  aop_bit_or = 0x10,
-  aop_bit_xor = 0x11,
-  aop_bit_not = 0x12,
-  aop_equal = 0x13,
-  aop_less_signed = 0x14,
-  aop_less_unsigned = 0x15,
-  aop_ext = 0x16,
-  aop_ref8 = 0x17,
-  aop_ref16 = 0x18,
-  aop_ref32 = 0x19,
-  aop_ref64 = 0x1a,
-  aop_ref_float = 0x1b,
-  aop_ref_double = 0x1c,
-  aop_ref_long_double = 0x1d,
-  aop_l_to_d = 0x1e,
-  aop_d_to_l = 0x1f,
-  aop_if_goto = 0x20,
-  aop_goto = 0x21,
-  aop_const8 = 0x22,
-  aop_const16 = 0x23,
-  aop_const32 = 0x24,
-  aop_const64 = 0x25,
-  aop_reg = 0x26,
-  aop_end = 0x27,
-  aop_dup = 0x28,
-  aop_pop = 0x29,
-  aop_zero_ext = 0x2a,
-  aop_swap = 0x2b,
-  aop_trace16 = 0x30,
-  aop_last
-};
+enum agent_op
+  {
+    aop_float = 0x01,
+    aop_add = 0x02,
+    aop_sub = 0x03,
+    aop_mul = 0x04,
+    aop_div_signed = 0x05,
+    aop_div_unsigned = 0x06,
+    aop_rem_signed = 0x07,
+    aop_rem_unsigned = 0x08,
+    aop_lsh = 0x09,
+    aop_rsh_signed = 0x0a,
+    aop_rsh_unsigned = 0x0b,
+    aop_trace = 0x0c,
+    aop_trace_quick = 0x0d,
+    aop_log_not = 0x0e,
+    aop_bit_and = 0x0f,
+    aop_bit_or = 0x10,
+    aop_bit_xor = 0x11,
+    aop_bit_not = 0x12,
+    aop_equal = 0x13,
+    aop_less_signed = 0x14,
+    aop_less_unsigned = 0x15,
+    aop_ext = 0x16,
+    aop_ref8 = 0x17,
+    aop_ref16 = 0x18,
+    aop_ref32 = 0x19,
+    aop_ref64 = 0x1a,
+    aop_ref_float = 0x1b,
+    aop_ref_double = 0x1c,
+    aop_ref_long_double = 0x1d,
+    aop_l_to_d = 0x1e,
+    aop_d_to_l = 0x1f,
+    aop_if_goto = 0x20,
+    aop_goto = 0x21,
+    aop_const8 = 0x22,
+    aop_const16 = 0x23,
+    aop_const32 = 0x24,
+    aop_const64 = 0x25,
+    aop_reg = 0x26,
+    aop_end = 0x27,
+    aop_dup = 0x28,
+    aop_pop = 0x29,
+    aop_zero_ext = 0x2a,
+    aop_swap = 0x2b,
+    aop_trace16 = 0x30,
+    aop_last
+  };
+\f
 
 
-\f
 /* Functions for building expressions.  */
 
 /* Allocate a new, empty agent expression.  */
@@ -141,21 +143,22 @@ extern struct agent_expr *new_agent_expr PARAMS ((CORE_ADDR));
 
 /* Free a agent expression.  */
 extern void free_agent_expr PARAMS ((struct agent_expr *));
+extern struct cleanup *make_cleanup_free_agent_expr (struct agent_expr *);
 
 /* Append a simple operator OP to EXPR.  */
-extern void ax_simple PARAMS ((struct agent_expr *EXPR, enum agent_op OP));
+extern void ax_simple PARAMS ((struct agent_expr * EXPR, enum agent_op OP));
 
 /* Append the floating-point prefix, for the next bytecode.  */
 #define ax_float(EXPR) (ax_simple ((EXPR), aop_float))
 
 /* Append a sign-extension instruction to EXPR, to extend an N-bit value.  */
-extern void ax_ext PARAMS ((struct agent_expr *EXPR, int N));
+extern void ax_ext PARAMS ((struct agent_expr * EXPR, int N));
 
 /* Append a zero-extension instruction to EXPR, to extend an N-bit value.  */
-extern void ax_zero_ext PARAMS ((struct agent_expr *EXPR, int N));
+extern void ax_zero_ext PARAMS ((struct agent_expr * EXPR, int N));
 
 /* Append a trace_quick instruction to EXPR, to record N bytes.  */
-extern void ax_trace_quick PARAMS ((struct agent_expr *EXPR, int N));
+extern void ax_trace_quick PARAMS ((struct agent_expr * EXPR, int N));
 
 /* Append a goto op to EXPR.  OP is the actual op (must be aop_goto or
    aop_if_goto).  We assume we don't know the target offset yet,
@@ -163,123 +166,126 @@ extern void ax_trace_quick PARAMS ((struct agent_expr *EXPR, int N));
    for the target, and return the offset in EXPR of that space, so we
    can backpatch it once we do know the target offset.  Use ax_label
    to do the backpatching.  */
-extern int ax_goto PARAMS ((struct agent_expr *EXPR, enum agent_op OP));
+extern int ax_goto PARAMS ((struct agent_expr * EXPR, enum agent_op OP));
 
 /* Suppose a given call to ax_goto returns some value PATCH.  When you
    know the offset TARGET that goto should jump to, call
-       ax_label (EXPR, PATCH, TARGET)
+   ax_label (EXPR, PATCH, TARGET)
    to patch TARGET into the ax_goto instruction.  */
-extern void ax_label PARAMS ((struct agent_expr *EXPR, int patch, int target));
+extern void ax_label PARAMS ((struct agent_expr * EXPR, int patch, int target));
 
 /* Assemble code to push a constant on the stack.  */
-extern void ax_const_l PARAMS ((struct agent_expr *EXPR, LONGEST l));
-extern void ax_const_d PARAMS ((struct agent_expr *EXPR, LONGEST d));
+extern void ax_const_l PARAMS ((struct agent_expr * EXPR, LONGEST l));
+extern void ax_const_d PARAMS ((struct agent_expr * EXPR, LONGEST d));
 
 /* Assemble code to push the value of register number REG on the
    stack.  */
-extern void ax_reg PARAMS ((struct agent_expr *EXPR, int REG));
-
+extern void ax_reg PARAMS ((struct agent_expr * EXPR, int REG));
 \f
+
 /* Functions for printing out expressions, and otherwise debugging
    things.  */
 
 /* Disassemble the expression EXPR, writing to F.  */
-extern void ax_print PARAMS ((GDB_FILE *f, struct agent_expr *EXPR));
+extern void ax_print (struct ui_file *f, struct agent_expr * EXPR);
 
 /* An entry in the opcode map.  */
-struct aop_map {
+struct aop_map
+  {
 
-  /* The name of the opcode.  Null means that this entry is not a
-     valid opcode --- a hole in the opcode space.  */
-  char *name;
+    /* The name of the opcode.  Null means that this entry is not a
+       valid opcode --- a hole in the opcode space.  */
+    char *name;
 
-  /* All opcodes take no operands from the bytecode stream, or take
-     unsigned integers of various sizes.  If this is a positive number
-     n, then the opcode is followed by an n-byte operand, which should
-     be printed as an unsigned integer.  If this is zero, then the
-     opcode takes no operands from the bytecode stream.
+    /* All opcodes take no operands from the bytecode stream, or take
+       unsigned integers of various sizes.  If this is a positive number
+       n, then the opcode is followed by an n-byte operand, which should
+       be printed as an unsigned integer.  If this is zero, then the
+       opcode takes no operands from the bytecode stream.
 
-     If we get more complicated opcodes in the future, don't add other
-     magic values of this; that's a crock.  Add an `enum encoding'
-     field to this, or something like that.  */
-  int op_size;
+       If we get more complicated opcodes in the future, don't add other
+       magic values of this; that's a crock.  Add an `enum encoding'
+       field to this, or something like that.  */
+    int op_size;
 
-  /* The size of the data operated upon, in bits, for bytecodes that
-     care about that (ref and const).  Zero for all others.  */
-  int data_size;
+    /* The size of the data operated upon, in bits, for bytecodes that
+       care about that (ref and const).  Zero for all others.  */
+    int data_size;
 
-  /* Number of stack elements consumed, and number produced.  */
-  int consumed, produced;
-};
+    /* Number of stack elements consumed, and number produced.  */
+    int consumed, produced;
+  };
 
 /* Map of the bytecodes, indexed by bytecode number.  */
 extern struct aop_map aop_map[];
 
 /* Different kinds of flaws an agent expression might have, as
    detected by agent_reqs.  */
-enum agent_flaws {
-  agent_flaw_none = 0,         /* code is good */
+enum agent_flaws
+  {
+    agent_flaw_none = 0,       /* code is good */
 
-  /* There is an invalid instruction in the stream.  */
-  agent_flaw_bad_instruction,
+    /* There is an invalid instruction in the stream.  */
+    agent_flaw_bad_instruction,
 
-  /* There is an incomplete instruction at the end of the expression.  */
-  agent_flaw_incomplete_instruction,
+    /* There is an incomplete instruction at the end of the expression.  */
+    agent_flaw_incomplete_instruction,
 
-  /* agent_reqs was unable to prove that every jump target is to a
-     valid offset.  Valid offsets are within the bounds of the
-     expression, and to a valid instruction boundary.  */
-  agent_flaw_bad_jump,
+    /* agent_reqs was unable to prove that every jump target is to a
+       valid offset.  Valid offsets are within the bounds of the
+       expression, and to a valid instruction boundary.  */
+    agent_flaw_bad_jump,
 
-  /* agent_reqs was unable to prove to its satisfaction that, for each
-     jump target location, the stack will have the same height whether
-     that location is reached via a jump or by straight execution.  */
-  agent_flaw_height_mismatch,
+    /* agent_reqs was unable to prove to its satisfaction that, for each
+       jump target location, the stack will have the same height whether
+       that location is reached via a jump or by straight execution.  */
+    agent_flaw_height_mismatch,
 
-  /* agent_reqs was unable to prove that every instruction following
-     an unconditional jump was the target of some other jump.  */
-  agent_flaw_hole
-};
+    /* agent_reqs was unable to prove that every instruction following
+       an unconditional jump was the target of some other jump.  */
+    agent_flaw_hole
+  };
 
 /* Structure describing the requirements of a bytecode expression.  */
-struct agent_reqs {
+struct agent_reqs
+  {
+
+    /* If the following is not equal to agent_flaw_none, the rest of the
+       information in this structure is suspect.  */
+    enum agent_flaws flaw;
 
-  /* If the following is not equal to agent_flaw_none, the rest of the
-     information in this structure is suspect.  */
-  enum agent_flaws flaw;
+    /* Number of elements left on stack at end; may be negative if expr
+       only consumes elements.  */
+    int final_height;
 
-  /* Number of elements left on stack at end; may be negative if expr
-     only consumes elements.  */
-  int final_height;
+    /* Maximum and minimum stack height, relative to initial height.  */
+    int max_height, min_height;
 
-  /* Maximum and minimum stack height, relative to initial height.  */
-  int max_height, min_height;
+    /* Largest `ref' or `const' opcode used, in bits.  Zero means the
+       expression has no such instructions.  */
+    int max_data_size;
 
-  /* Largest `ref' or `const' opcode used, in bits.  Zero means the
-     expression has no such instructions.  */
-  int max_data_size;
+    /* Bit vector of registers used.  Register R is used iff
 
-  /* Bit vector of registers used.  Register R is used iff
-  
-       reg_mask[R / 8] & (1 << (R % 8))
+       reg_mask[R / 8] & (1 << (R % 8))
 
-     is non-zero.  Note!  You may not assume that this bitmask is long
-     enough to hold bits for all the registers of the machine; the
-     agent expression code has no idea how many registers the machine
-     has.  However, the bitmask is reg_mask_len bytes long, so the
-     valid register numbers run from 0 to reg_mask_len * 8 - 1.  
+       is non-zero.  Note!  You may not assume that this bitmask is long
+       enough to hold bits for all the registers of the machine; the
+       agent expression code has no idea how many registers the machine
+       has.  However, the bitmask is reg_mask_len bytes long, so the
+       valid register numbers run from 0 to reg_mask_len * 8 - 1.  
 
-     We're assuming eight-bit bytes.  So sue me.
+       We're assuming eight-bit bytes.  So sue me.
 
-     The caller should free reg_list when done.  */
-  int reg_mask_len;
-  unsigned char *reg_mask;
-};
+       The caller should free reg_list when done.  */
+    int reg_mask_len;
+    unsigned char *reg_mask;
+  };
 
 
 /* Given an agent expression AX, fill in an agent_reqs structure REQS
    describing it.  */
-extern void ax_reqs PARAMS ((struct agent_expr *ax,
-                            struct agent_reqs *reqs));
+extern void ax_reqs PARAMS ((struct agent_expr * ax,
+                            struct agent_reqs * reqs));
 
 #endif /* AGENTEXPR_H */
index 58db9b3..eefd556 100644 (file)
@@ -999,7 +999,7 @@ validate_actionline (line, t)
          /* we have something to collect, make sure that the expr to
             bytecode translator can handle it and that it's not too long */
          aexpr = gen_trace_for_expr (t->address, exp);
-         (void) make_cleanup ((make_cleanup_func) free_agent_expr, aexpr);
+         make_cleanup_free_agent_expr (aexpr);
 
          if (aexpr->len > MAX_AGENT_EXPR_LEN)
            error ("expression too complicated, try simplifying");
@@ -1617,8 +1617,7 @@ encode_actions (t, tdp_actions, stepping_actions)
                    default:    /* full-fledged expression */
                      aexpr = gen_trace_for_expr (t->address, exp);
 
-                     old_chain1 = make_cleanup ((make_cleanup_func)
-                                                free_agent_expr, aexpr);
+                     old_chain1 = make_cleanup_free_agent_expr (aexpr);
 
                      ax_reqs (aexpr, &areqs);
                      if (areqs.flaw != agent_flaw_none)