aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2023-06-20 11:09:18 -0600
committerTom Tromey <tom@tromey.com>2023-06-20 11:21:51 -0600
commit3ba0581955632c4b269bbcf8a458daafc9d167a6 (patch)
tree18715438c105befa4fbdb2142a2a3d3c8823b2bb
parent1e0e3ecd67156ea8a7ecb9e4fb709caa6004bd52 (diff)
downloadbinutils-3ba0581955632c4b269bbcf8a458daafc9d167a6.zip
binutils-3ba0581955632c4b269bbcf8a458daafc9d167a6.tar.gz
binutils-3ba0581955632c4b269bbcf8a458daafc9d167a6.tar.bz2
Remove aop_last
aop_last is only used for an assertion. However, due to the '.def' construct in the sources, this assert could never plausibly trigger (the assert predates the use of a .def file here). This patch removes the constant and the assert. Reviewed-by: John Baldwin <jhb@FreeBSD.org>
-rw-r--r--gdb/ax-general.c6
-rw-r--r--gdb/ax.h1
2 files changed, 0 insertions, 7 deletions
diff --git a/gdb/ax-general.c b/gdb/ax-general.c
index a8451d7..4ace2b4 100644
--- a/gdb/ax-general.c
+++ b/gdb/ax-general.c
@@ -349,12 +349,6 @@ ax_print (struct ui_file *f, struct agent_expr *x)
}
gdb_printf (f, _("\n"));
- /* Check the size of the name array against the number of entries in
- the enum, to catch additions that people didn't sync. */
- 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->buf.size ();)
{
enum agent_op op = (enum agent_op) x->buf[i];
diff --git a/gdb/ax.h b/gdb/ax.h
index 1fdecb2..e02aa71 100644
--- a/gdb/ax.h
+++ b/gdb/ax.h
@@ -154,7 +154,6 @@ enum agent_op
aop_ ## NAME = VALUE,
#include "gdbsupport/ax.def"
#undef DEFOP
- aop_last
};