]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
sim: use ARRAY_SIZE instead of ad-hoc sizeof calculations
authorMike Frysinger <vapier@gentoo.org>
Mon, 13 Feb 2017 05:12:35 +0000 (00:12 -0500)
committerMike Frysinger <vapier@gentoo.org>
Mon, 13 Feb 2017 06:26:21 +0000 (01:26 -0500)
39 files changed:
sim/arm/ChangeLog
sim/arm/armos.c
sim/arm/armsupp.c
sim/arm/wrapper.c
sim/cris/ChangeLog
sim/cris/decodev10.c
sim/cris/decodev32.c
sim/cris/sim-if.c
sim/frv/ChangeLog
sim/frv/decode.c
sim/iq2000/ChangeLog
sim/iq2000/decode.c
sim/lm32/ChangeLog
sim/lm32/decode.c
sim/m32c/ChangeLog
sim/m32c/srcdest.c
sim/m32r/ChangeLog
sim/m32r/decode.c
sim/m32r/decode2.c
sim/m32r/decodex.c
sim/m68hc11/ChangeLog
sim/m68hc11/gencode.c
sim/m68hc11/interrupts.c
sim/ppc/ChangeLog
sim/ppc/cpu.h
sim/ppc/emul_bugapi.c
sim/ppc/emul_generic.h
sim/ppc/emul_netbsd.c
sim/ppc/emul_unix.c
sim/ppc/options.c
sim/ppc/ppc-instructions
sim/rx/ChangeLog
sim/rx/rx.c
sim/sh/ChangeLog
sim/sh/gencode.c
sim/sh/interp.c
sim/sh64/ChangeLog
sim/sh64/decode-compact.c
sim/sh64/decode-media.c

index 2ce94292420c0fc87d48607c3960d53e8b57dfd7..488178a5df93a31ec45eb9fb40f13c270a94300a 100644 (file)
@@ -1,3 +1,11 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * armos.c: Include libiberty.h.
+       (SWIopen): Use ARRAY_SIZE.
+       * armsupp.c: Include libiberty.h.
+       (ModeToBank): Use ARRAY_SIZE.
+       * wrapper.c (sim_target_parse_command_line): Likewise.
+
 2016-07-14  Nick Clifton  <nickc@redhat.com>
 
        * armemu.c (Multiply64): Only issue error messages about invalid
index ea3d22979ac8d001ddcab507000fd610bdd0e95b..76c449b73d0f7dd4b69fe32de97056fb354c0efd 100644 (file)
@@ -23,6 +23,7 @@
 
 #include "config.h"
 #include "ansidecl.h"
+#include "libiberty.h"
 
 #include <time.h>
 #include <errno.h>
@@ -260,7 +261,7 @@ SWIopen (ARMul_State * state, ARMword name, ARMword SWIflags)
     return;
 
   /* Now we need to decode the Demon open mode.  */
-  if (SWIflags >= sizeof (translate_open_mode) / sizeof (translate_open_mode[0]))
+  if (SWIflags >= ARRAY_SIZE (translate_open_mode))
     flags = 0;
   else
     flags = translate_open_mode[SWIflags];
index 3ff4bf9a6b0536de2e02b55fc0235634dfeb5c49..11bb53c5b7049130f201acc581f21db0f51d2765 100644 (file)
@@ -17,6 +17,7 @@
 #include "armdefs.h"
 #include "armemu.h"
 #include "ansidecl.h"
+#include "libiberty.h"
 #include <math.h>
 
 /* Definitions for the support routines.  */
@@ -373,7 +374,7 @@ ModeToBank (ARMword mode)
     DUMMYBANK, DUMMYBANK, DUMMYBANK, SYSTEMBANK
   };
 
-  if (mode >= (sizeof (bankofmode) / sizeof (bankofmode[0])))
+  if (mode >= ARRAY_SIZE (bankofmode))
     return DUMMYBANK;
 
   return bankofmode[mode];
index efbeef943bcecb5174f043e1b6553a1efbcfd359..01f88d11e57c07600115fcf95f2412e06a46ed12 100644 (file)
@@ -740,7 +740,7 @@ sim_target_parse_command_line (int argc, char ** argv)
        {
          int i;
 
-         for (i = sizeof options / sizeof options[0]; i--;)
+         for (i = ARRAY_SIZE (options); i--;)
            if (strncmp (ptr, options[i].swi_option,
                         strlen (options[i].swi_option)) == 0)
              {
index ef96ec7faa0fd1ad4a0e2cfa7b3ca5259eacd6f5..1257a9e4fe5c722c14980b749641e25a7540b704 100644 (file)
@@ -1,3 +1,9 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decodev10.c (crisv10f_init_idesc_table): Use ARRAY_SIZE.
+       * decodev32.c (crisv32f_init_idesc_table): Likewise.
+       * sim-if.c (sim_open): Likewise.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index c4c174f3c40591ee74ac49ce379cd1049fd4da4a..d9242e35415c7b59b4645f79bb446e9e982361f9 100644 (file)
@@ -302,7 +302,7 @@ crisv10f_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = crisv10f_insn_sem, tend = t + sizeof (crisv10f_insn_sem) / sizeof (*t);
+  for (t = crisv10f_insn_sem, tend = t + ARRAY_SIZE (crisv10f_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 4ea146d2501d0c5eaafdb14d8758a008b7b6c2ce..10162e883690337e6bad09a5b3712cac8b9f50ed 100644 (file)
@@ -306,7 +306,7 @@ crisv32f_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = crisv32f_insn_sem, tend = t + sizeof (crisv32f_insn_sem) / sizeof (*t);
+  for (t = crisv32f_insn_sem, tend = t + ARRAY_SIZE (crisv32f_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 8e3b6677fa1a889f9ab45b8d55dcae955cab8e64..2ccb5289bdb48d1a47363005271c024d70a40be0 100644 (file)
@@ -786,7 +786,7 @@ sim_open (SIM_OPEN_KIND kind, host_callback *callback, struct bfd *abfd,
       /* Note that the linux kernel does not correctly compute the storage
         needs for the static-exe AUX vector.  */
 
-      csp -= sizeof (auxv_entries) / sizeof (auxv_entries[0]) * 4 * 2;
+      csp -= ARRAY_SIZE (auxv_entries) * 4 * 2;
 
       csp -= (envc + 1) * 4;
       csp -= (my_argc + 1) * 4;
@@ -874,7 +874,7 @@ sim_open (SIM_OPEN_KIND kind, host_callback *callback, struct bfd *abfd,
        goto abandon_chip;
 
       if (bfd_get_flavour (abfd) == bfd_target_elf_flavour)
-       for (i = 0; i < sizeof (auxv_entries) / sizeof (auxv_entries[0]); i++)
+       for (i = 0; i < ARRAY_SIZE (auxv_entries); i++)
          {
            write_dword (csp, auxv_entries[i].id);
            write_dword (csp + 4,
index 1642a26d0539e5cf118da4baf829d6a3b76ab720..68bf3886dbedcf697b30ff3d40301a6ea76e4c72 100644 (file)
@@ -1,3 +1,7 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decode.c (frvbf_init_idesc_table): Use ARRAY_SIZE.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index 6ce49a8d8655426899e3b36a368e8c6e13448024..69c155ead61f86467d5d6562e6dd3babcd98dcb3 100644 (file)
@@ -841,7 +841,7 @@ frvbf_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = frvbf_insn_sem, tend = t + sizeof (frvbf_insn_sem) / sizeof (*t);
+  for (t = frvbf_insn_sem, tend = t + ARRAY_SIZE (frvbf_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 5e91cbd2243ad13fc463ceaded400a45b115dafd..e2290ec02969c5396e692ab9404f106657aa65f4 100644 (file)
@@ -1,3 +1,7 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decode.c (iq2000bf_init_idesc_table): Use ARRAY_SIZE.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index 7d6bd2d911a4e0aa6f15a3439d555139a16aecf2..d3d4dfa972808989aee85e56fa847b2c970871d8 100644 (file)
@@ -242,7 +242,7 @@ iq2000bf_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = iq2000bf_insn_sem, tend = t + sizeof (iq2000bf_insn_sem) / sizeof (*t);
+  for (t = iq2000bf_insn_sem, tend = t + ARRAY_SIZE (iq2000bf_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 51869baa1b144e012e65e3a6cd4682ea8e18ac47..b724a46b3f57869366d619b23f83ae4a762bc878 100644 (file)
@@ -1,3 +1,7 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decode.c (lm32bf_init_idesc_table): Use ARRAY_SIZE.
+
 2016-08-15  Mike Frysinger  <vapier@gentoo.org>
 
        * sim-if.c (find_limit): Change prototype to take a SIM_DESC.
index 207c9c7386de45875e9d2dc52f76ed7af5d15a6b..52e7d8428129564662f0eab0b4e8ceed4c0fbf2a 100644 (file)
@@ -160,7 +160,7 @@ lm32bf_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = lm32bf_insn_sem, tend = t + sizeof (lm32bf_insn_sem) / sizeof (*t);
+  for (t = lm32bf_insn_sem, tend = t + ARRAY_SIZE (lm32bf_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 4205507addcb6f8281ccbfc12e0aeca50301bf3c..8f62fc30f9071cce6d2b5cb78e7026bd6d9fef15 100644 (file)
@@ -1,3 +1,8 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * srcdest.c: Include libiberty.h.
+       (decode_sd23): Use ARRAY_SIZE.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index 8dfe9d0f19ce129308b7b8e178c7b66fac44010e..937779b442eee233fd395f35a21a6eb272582d26 100644 (file)
@@ -22,6 +22,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 #include <stdio.h>
 #include <stdlib.h>
 
+#include "libiberty.h"
 #include "cpu.h"
 #include "mem.h"
 
@@ -354,7 +355,7 @@ decode_sd23 (int bbb, int bb, int bytes, int ind, int add)
   srcdest sd;
   int code = (bbb << 2) | bb;
 
-  if (code >= sizeof (modes23) / sizeof (modes23[0]))
+  if (code >= ARRAY_SIZE (modes23))
     abort ();
 
   if (trace)
index a0be81e29642572decf861c0e87f9b8aed67428f..6a9a85c6650c5fb55928556981b246834b916561 100644 (file)
@@ -1,3 +1,9 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decode.c (m32rbf_init_idesc_table): Use ARRAY_SIZE.
+       * decode2.c (m32r2f_init_idesc_table): Likewise.
+       * decodex.c (m32rxf_init_idesc_table): Likewise.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index 9f7998e0a45df0f0a6994413c2f979d01c08fc11..70494bccef85ce5aefc10f43a16e204fedd8f61d 100644 (file)
@@ -202,7 +202,7 @@ m32rbf_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = m32rbf_insn_sem, tend = t + sizeof (m32rbf_insn_sem) / sizeof (*t);
+  for (t = m32rbf_insn_sem, tend = t + ARRAY_SIZE (m32rbf_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index a8833dbb8f09e2bbaa1bf60a4408d6a7557c9395..f138d7cb21d04ccea2102f17ce96593fd2f7ba14 100644 (file)
@@ -235,7 +235,7 @@ m32r2f_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = m32r2f_insn_sem, tend = t + sizeof (m32r2f_insn_sem) / sizeof (*t);
+  for (t = m32r2f_insn_sem, tend = t + ARRAY_SIZE (m32r2f_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 4274b523ec57eb143c5cec8bfd7c9332b6ee7ac2..0b695cc28ce9775cdf46e80c7ea1f0118e8ddb88 100644 (file)
@@ -228,7 +228,7 @@ m32rxf_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = m32rxf_insn_sem, tend = t + sizeof (m32rxf_insn_sem) / sizeof (*t);
+  for (t = m32rxf_insn_sem, tend = t + ARRAY_SIZE (m32rxf_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index bc69d846b21c832d09ff57798e68b1c8f378e253..8947c4b7bc2a90ba11099c3975f6acf3388653df 100644 (file)
@@ -1,3 +1,12 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * gencode.c: Include libiberty.h.
+       (TABLE_SIZE): Delete.
+       (find_opcode_pattern): Change TABLE_SIZE to ARRAY_SIZE.
+       (gen_interpreter): Likewise.
+       * interrupts.c (TableSize): Delete.
+       (interrupts_update_pending): Change TableSize to ARRAY_SIZE.
+
 2016-08-16  Mike Frysinger  <vapier@gentoo.org>
 
        * sim-main.h (inline): Delete define.
index e207606cb36a4edd4567821185d646a606586cbb..3e85c9341843d36586c57899119fc3484c267e48 100644 (file)
@@ -24,10 +24,9 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 #include <errno.h>
 
 #include "ansidecl.h"
+#include "libiberty.h"
 #include "opcode/m68hc11.h"
 
-#define TABLE_SIZE(X)      (sizeof(X) / sizeof(X[0]))
-
 /* Combination of CCR flags.  */
 #define M6811_ZC_BIT   M6811_Z_BIT|M6811_C_BIT
 #define M6811_NZ_BIT   M6811_N_BIT|M6811_Z_BIT
@@ -1812,7 +1811,7 @@ find_opcode_pattern (const struct m6811_opcode_def *opcode)
     {
       pattern = opcode->name;
     }
-  for (i = 0; i < TABLE_SIZE(m6811_opcode_patterns); i++)
+  for (i = 0; i < ARRAY_SIZE (m6811_opcode_patterns); i++)
     {
       if (strcmp (m6811_opcode_patterns[i].name, pattern) == 0)
        {
@@ -2036,13 +2035,13 @@ gen_interpreter (FILE *fp)
 {
   int col = 0;
 
-  prepare_table (m6811_page1_opcodes, TABLE_SIZE (m6811_page1_opcodes));
-  prepare_table (m6811_page2_opcodes, TABLE_SIZE (m6811_page2_opcodes));
-  prepare_table (m6811_page3_opcodes, TABLE_SIZE (m6811_page3_opcodes));
-  prepare_table (m6811_page4_opcodes, TABLE_SIZE (m6811_page4_opcodes));
+  prepare_table (m6811_page1_opcodes, ARRAY_SIZE (m6811_page1_opcodes));
+  prepare_table (m6811_page2_opcodes, ARRAY_SIZE (m6811_page2_opcodes));
+  prepare_table (m6811_page3_opcodes, ARRAY_SIZE (m6811_page3_opcodes));
+  prepare_table (m6811_page4_opcodes, ARRAY_SIZE (m6811_page4_opcodes));
 
-  prepare_table (m6812_page1_opcodes, TABLE_SIZE (m6812_page1_opcodes));
-  prepare_table (m6812_page2_opcodes, TABLE_SIZE (m6812_page2_opcodes));
+  prepare_table (m6812_page1_opcodes, ARRAY_SIZE (m6812_page1_opcodes));
+  prepare_table (m6812_page2_opcodes, ARRAY_SIZE (m6812_page2_opcodes));
 
   /* Generate header of interpretor.  */
   print (fp, col, "/* File generated automatically by gencode. */\n");
@@ -2051,25 +2050,25 @@ gen_interpreter (FILE *fp)
   if (cpu_type & cpu6811)
     {
       gen_cycle_table (fp, "cycles_page1", m6811_page1_opcodes,
-                      TABLE_SIZE (m6811_page1_opcodes));
+                      ARRAY_SIZE (m6811_page1_opcodes));
       gen_cycle_table (fp, "cycles_page2", m6811_page2_opcodes,
-                      TABLE_SIZE (m6811_page2_opcodes));
+                      ARRAY_SIZE (m6811_page2_opcodes));
       gen_cycle_table (fp, "cycles_page3", m6811_page3_opcodes,
-                      TABLE_SIZE (m6811_page3_opcodes));
+                      ARRAY_SIZE (m6811_page3_opcodes));
       gen_cycle_table (fp, "cycles_page4", m6811_page4_opcodes,
-                      TABLE_SIZE (m6811_page4_opcodes));
+                      ARRAY_SIZE (m6811_page4_opcodes));
 
       gen_function_entry (fp, "static void\ncpu_page3_interp", 0);
       gen_interpreter_for_table (fp, indent_level,
                                 m6811_page3_opcodes,
-                                TABLE_SIZE(m6811_page3_opcodes),
+                                ARRAY_SIZE (m6811_page3_opcodes),
                                 "cycles_page3");
       gen_function_close (fp);
   
       gen_function_entry (fp, "static void\ncpu_page4_interp", 0);
       gen_interpreter_for_table (fp, indent_level,
                                 m6811_page4_opcodes,
-                                TABLE_SIZE(m6811_page4_opcodes),
+                                ARRAY_SIZE (m6811_page4_opcodes),
                                 "cycles_page4");
       gen_function_close (fp);
 
@@ -2078,7 +2077,7 @@ gen_interpreter (FILE *fp)
                           USE_SRC8 | USE_DST8);
       gen_interpreter_for_table (fp, indent_level,
                                 m6811_page2_opcodes,
-                                TABLE_SIZE(m6811_page2_opcodes),
+                                ARRAY_SIZE (m6811_page2_opcodes),
                                 "cycles_page2");
       gen_function_close (fp);
 
@@ -2087,22 +2086,22 @@ gen_interpreter (FILE *fp)
                           USE_SRC8 | USE_DST8);
 
       gen_interpreter_for_table (fp, indent_level, m6811_page1_opcodes,
-                                TABLE_SIZE(m6811_page1_opcodes),
+                                ARRAY_SIZE (m6811_page1_opcodes),
                                 "cycles_page1");
       gen_function_close (fp);
     }
   else
     {
       gen_cycle_table (fp, "cycles_page1", m6812_page1_opcodes,
-                      TABLE_SIZE (m6812_page1_opcodes));
+                      ARRAY_SIZE (m6812_page1_opcodes));
       gen_cycle_table (fp, "cycles_page2", m6812_page2_opcodes,
-                      TABLE_SIZE (m6812_page2_opcodes));
+                      ARRAY_SIZE (m6812_page2_opcodes));
 
       gen_function_entry (fp, "static void\ncpu_page2_interp",
                           USE_SRC8 | USE_DST8);
       gen_interpreter_for_table (fp, indent_level,
                                 m6812_page2_opcodes,
-                                TABLE_SIZE(m6812_page2_opcodes),
+                                ARRAY_SIZE (m6812_page2_opcodes),
                                 "cycles_page2");
       gen_function_close (fp);
 
@@ -2111,7 +2110,7 @@ gen_interpreter (FILE *fp)
                           USE_SRC8 | USE_DST8);
 
       gen_interpreter_for_table (fp, indent_level, m6812_page1_opcodes,
-                                TABLE_SIZE(m6812_page1_opcodes),
+                                ARRAY_SIZE (m6812_page1_opcodes),
                                 "cycles_page1");
       gen_function_close (fp);
     }
index 129f3c022aa0551defa73e6306b0c121f5d81727..ed826556ca40156702b74b44b0742e0d28509a1a 100644 (file)
@@ -91,7 +91,6 @@ struct interrupt_def idefs[] = {
 #endif
 };
 
-#define TableSize(X) (sizeof X / sizeof(X[0]))
 #define CYCLES_MAX ((((signed64) 1) << 62) - 1)
 
 enum
@@ -290,7 +289,7 @@ interrupts_update_pending (struct interrupts *interrupts)
   set_mask = 0;
   ioregs = &interrupts->cpu->ios[0];
   
-  for (i = 0; i < TableSize(idefs); i++)
+  for (i = 0; i < ARRAY_SIZE (idefs); i++)
     {
       struct interrupt_def *idef = &idefs[i];
       uint8 data;
index 7ed024e88b32217cd0a55cc15e5a57f8a8586587..c0bb1f5b8b4611dea8888ee9e6a5312aeafadfdc 100644 (file)
@@ -1,3 +1,15 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * cpu.h: Include libiberty.h.
+       * emul_bugapi.c (emul_bugapi_instruction_name): Use ARRAY_SIZE.
+       * emul_generic.h: Include libiberty.h.
+       * emul_netbsd.c (emul_netbsd_syscalls): Use ARRAY_SIZE.
+       * emul_unix.c (convert_to_solaris_stat): Likewise.
+       (emul_solaris_syscalls): Likewise.
+       (emul_linux_syscalls): Likewise.
+       * options.c (print_options): Likewise.
+       * ppc-instructions: Likewise.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * configure.ac (sim-assert): Call AC_MSG_CHECKING,
index cb141f20657a3eac53da063018814963c036c671..bafb5e1791a1f23fccf61d4e6f6afcc06f2d7ab0 100644 (file)
@@ -34,6 +34,7 @@
 #include "os_emul.h"
 #include "mon.h"
 #include "model.h"
+#include "libiberty.h"
 
 #ifndef CONST_ATTRIBUTE
 #define CONST_ATTRIBUTE __attribute__((__const__))
index e33d0cf2995d85d3c9d624087c471ddba4d6f9fd..8481b6fb39e6cfcad9496507caf10ded4532aef7 100644 (file)
@@ -339,7 +339,7 @@ emul_bugapi_instruction_name(int call_id)
   static char buffer[40];
   int i;
 
-  for (i = 0; i < sizeof (bug_mapping) / sizeof (bug_mapping[0]); i++)
+  for (i = 0; i < ARRAY_SIZE (bug_mapping); i++)
     {
       if (bug_mapping[i].value == call_id)
        return bug_mapping[i].info;
index e697abddbb1d9cd913e755321935c9c5375d3105..9ea23d4cbd551ee0a2344767d103a94bf4837fb4 100644 (file)
@@ -28,6 +28,7 @@
 #include "tree.h"
 
 #include "bfd.h"
+#include "libiberty.h"
 
 #ifndef INLINE_EMUL_GENERIC
 #define INLINE_EMUL_GENERIC
index 12dfb21b3b0697b63c0a0661f7f287c7ec79db70..9b80fc25dc812ab4317de3a7481b34a31199cc07 100644 (file)
@@ -1413,11 +1413,11 @@ static char *(netbsd_signal_names[]) = {
 
 static emul_syscall emul_netbsd_syscalls = {
   netbsd_descriptors,
-  sizeof(netbsd_descriptors) / sizeof(netbsd_descriptors[0]),
+  ARRAY_SIZE (netbsd_descriptors),
   netbsd_error_names,
-  sizeof(netbsd_error_names) / sizeof(netbsd_error_names[0]),
+  ARRAY_SIZE (netbsd_error_names),
   netbsd_signal_names,
-  sizeof(netbsd_signal_names) / sizeof(netbsd_signal_names[0]),
+  ARRAY_SIZE (netbsd_signal_names),
 };
 
 
index 1475474727eaa452b706f53bf8644b6943c78d6f..3a1c24586fa2ad3d699d3989af464f3c71b1588f 100644 (file)
@@ -1147,15 +1147,15 @@ convert_to_solaris_stat(unsigned_word addr,
   target.st_mtim.tv_sec  = H2T_4(host->st_mtime);
   target.st_mtim.tv_usec = 0;
 
-  for (i = 0; i < sizeof (target.st_pad1) / sizeof (target.st_pad1[0]); i++)
+  for (i = 0; i < ARRAY_SIZE (target.st_pad1); i++)
     target.st_pad1[i] = 0;
 
-  for (i = 0; i < sizeof (target.st_pad2) / sizeof (target.st_pad2[0]); i++)
+  for (i = 0; i < ARRAY_SIZE (target.st_pad2); i++)
     target.st_pad2[i] = 0;
 
   target.st_pad3 = 0;
 
-  for (i = 0; i < sizeof (target.st_pad4) / sizeof (target.st_pad4[0]); i++)
+  for (i = 0; i < ARRAY_SIZE (target.st_pad4); i++)
     target.st_pad4[i] = 0;
 
   /* For now, just punt and always say it is a ufs file */
@@ -1945,11 +1945,11 @@ static char *(solaris_signal_names[]) = {
 
 static emul_syscall emul_solaris_syscalls = {
   solaris_descriptors,
-  sizeof(solaris_descriptors) / sizeof(solaris_descriptors[0]),
+  ARRAY_SIZE (solaris_descriptors),
   solaris_error_names,
-  sizeof(solaris_error_names) / sizeof(solaris_error_names[0]),
+  ARRAY_SIZE (solaris_error_names),
   solaris_signal_names,
-  sizeof(solaris_signal_names) / sizeof(solaris_signal_names[0]),
+  ARRAY_SIZE (solaris_signal_names),
 };
 
 
@@ -2824,11 +2824,11 @@ static char *(linux_signal_names[]) = {
 
 static emul_syscall emul_linux_syscalls = {
   linux_descriptors,
-  sizeof(linux_descriptors) / sizeof(linux_descriptors[0]),
+  ARRAY_SIZE (linux_descriptors),
   linux_error_names,
-  sizeof(linux_error_names) / sizeof(linux_error_names[0]),
+  ARRAY_SIZE (linux_error_names),
   linux_signal_names,
-  sizeof(linux_signal_names) / sizeof(linux_signal_names[0]),
+  ARRAY_SIZE (linux_signal_names),
 };
 
 
index 169378ef93dc540bcd9edd91838b2a31008fbf73..ddb492f54fb7095ef2d1952da56a700e9f76de39 100644 (file)
@@ -216,7 +216,7 @@ print_options (void)
     int max_len = 0;
     int cols;
 
-    for (i = 0; i < sizeof (defines) / sizeof (defines[0]); i++) {
+    for (i = 0; i < ARRAY_SIZE (defines); i++) {
       int len = strlen (defines[i]);
       if (len > max_len)
        max_len = len;
@@ -227,10 +227,10 @@ print_options (void)
       cols = 1;
 
     printf_filtered ("\n#defines:");
-    for (i = 0; i < sizeof (defines) / sizeof (defines[0]); i++) {
+    for (i = 0; i < ARRAY_SIZE (defines); i++) {
       const char *const prefix = ((i % cols) == 0) ? "\n" : "";
       printf_filtered ("%s  %s%*s", prefix, defines[i],
-                      (((i == (sizeof (defines) / sizeof (defines[0])) - 1)
+                      (((i == ARRAY_SIZE (defines) - 1)
                         || (((i + 1) % cols) == 0))
                        ? 0
                        : max_len + 4 - strlen (defines[i])),
index 1a2e51afaf2a7bf79c501a61016b67487fcd5767..5e6d21df561d7cccf704628a783fefcd8645b893 100644 (file)
@@ -908,7 +908,7 @@ model_print *::model-function::model_mon_info:model_data *model_ptr
          tail->suffix_singular = "";
        }
 
-       for (j = 0; j < (sizeof(ppc_branch_conditional_name) / sizeof(ppc_branch_conditional_name[0])) ; j++) {
+       for (j = 0; j < ARRAY_SIZE (ppc_branch_conditional_name); j++) {
          if (model_ptr->nr_branch_conditional[j]) {
            tail->next = ZALLOC(model_print);
            tail = tail->next;
index 39de491df9f891d7d1b27a33fefb1716d6a0ccd0..d29e429feddf0fa8f72f10eb92ff8ac62244f7a7 100644 (file)
@@ -1,3 +1,8 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * rx.c: Include libiberty.h.
+       (N_RXO, N_RXT): Use ARRAY_SIZE.
+
 2016-07-27  Alan Modra  <amodra@gmail.com>
 
        * load.c: Don't include libbfd.h.
index 82b92b5167d3abe0b63382f669329265ee6e6c24..28dd2b638d1071b2b3aa4557dedb9dbaa7dec9de 100644 (file)
@@ -23,6 +23,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 #include <stdlib.h>
 #include <string.h>
 #include <signal.h>
+#include "libiberty.h"
 
 #include "opcode/rx.h"
 #include "cpu.h"
@@ -151,8 +152,8 @@ static const char * optype_names[] = {
   "RbRi"       /* [Rb + scale * Ri] */
 };
 
-#define N_RXO (sizeof(id_names)/sizeof(id_names[0]))
-#define N_RXT (sizeof(optype_names)/sizeof(optype_names[0]))
+#define N_RXO ARRAY_SIZE (id_names)
+#define N_RXT ARRAY_SIZE (optype_names)
 #define N_MAP 90
 
 static unsigned long long benchmark_start_cycle;
index 9d3cf44132d6f82eb6b171202be75d10259e1d10..6430e6db2094cc6204bbf097c035b7ca3544245d 100644 (file)
@@ -1,3 +1,9 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * gencode.c: Include libiberty.h.
+       (conflict_warn): Use ARRAY_SIZE.
+       * interp.c (init_dsp): Likewise.
+
 2016-04-10  Oleg Endo  <olegendo@gcc.gnu.org>
 
        * interp.c (dmul): Split into dmul_s and dmul_u.  Use explicit integer
index 2f1a3f5be28d168bab12beeb36f86b962c06a736..77a83d637685066f80a9fb23e8cd80e05e5152b7 100644 (file)
@@ -35,6 +35,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include "libiberty.h"
 
 #define MAX_NR_STUFF 42
 
@@ -2595,7 +2596,7 @@ conflict_warn (int val, int i)
   fprintf (stderr, "Warning: opcode table conflict: 0x%04x (idx %d && %d)\n",
           val, i, table[val]);
 
-  for (ix = sizeof (tab) / sizeof (tab[0]); ix >= 0; ix--)
+  for (ix = ARRAY_SIZE (tab); ix >= 0; ix--)
     if (tab[ix].index == i || tab[ix].index == j)
       {
        key = ((tab[ix].code[0] - '0') << 3) + 
@@ -2607,7 +2608,7 @@ conflict_warn (int val, int i)
          fprintf (stderr, "  %s -- %s\n", tab[ix].code, tab[ix].name);
       }
 
-  for (ix = sizeof (movsxy_tab) / sizeof (movsxy_tab[0]); ix >= 0; ix--)
+  for (ix = ARRAY_SIZE (movsxy_tab); ix >= 0; ix--)
     if (movsxy_tab[ix].index == i || movsxy_tab[ix].index == j)
       {
        key = ((movsxy_tab[ix].code[0] - '0') << 3) + 
@@ -2620,7 +2621,7 @@ conflict_warn (int val, int i)
                   movsxy_tab[ix].code, movsxy_tab[ix].name);
       }
 
-  for (ix = sizeof (ppi_tab) / sizeof (ppi_tab[0]); ix >= 0; ix--)
+  for (ix = ARRAY_SIZE (ppi_tab); ix >= 0; ix--)
     if (ppi_tab[ix].index == i || ppi_tab[ix].index == j)
       {
        key = ((ppi_tab[ix].code[0] - '0') << 3) + 
index 32618ba4490232f04eb8544c81f7e3eefbce27a9..c77cf3b1d3bf10e38be373542a4db57e5c4cb2c0 100644 (file)
@@ -1658,7 +1658,7 @@ init_dsp (struct bfd *abfd)
     {
       int i, tmp;
 
-      for (i = (sizeof sh_dsp_table / sizeof sh_dsp_table[0]) - 1; i >= 0; i--)
+      for (i = ARRAY_SIZE (sh_dsp_table) - 1; i >= 0; i--)
        {
          tmp = sh_jump_table[0xf000 + i];
          sh_jump_table[0xf000 + i] = sh_dsp_table[i];
index 7627e21caf0799f21944ddb987a591cd9082ed37..c0d31dd35238f381eb779892997fb82286d86109 100644 (file)
@@ -1,3 +1,8 @@
+2017-02-13  Mike Frysinger  <vapier@gentoo.org>
+
+       * decode-compact.c (sh64_compact_init_idesc_table): Use ARRAY_SIZE.
+       * decode-media.c (sh64_media_init_idesc_table): Likewise.
+
 2016-01-10  Mike Frysinger  <vapier@gentoo.org>
 
        * config.in, configure: Regenerate.
index 3b231666f853d376d7eeb5ab711c757fd0e0a9ba..0eeabab75037c6957ce94b6e39b5b94c3b091542 100644 (file)
@@ -292,7 +292,7 @@ sh64_compact_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = sh64_compact_insn_sem, tend = t + sizeof (sh64_compact_insn_sem) / sizeof (*t);
+  for (t = sh64_compact_insn_sem, tend = t + ARRAY_SIZE (sh64_compact_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);
index 40aa9836de759936a60ee4960ea0609c0cc29fa3..74fb83dbe72de179236747d0396664dda4c67093 100644 (file)
@@ -306,7 +306,7 @@ sh64_media_init_idesc_table (SIM_CPU *cpu)
     init_idesc (cpu, id, t);
 
   /* Now fill in the values for the chosen cpu.  */
-  for (t = sh64_media_insn_sem, tend = t + sizeof (sh64_media_insn_sem) / sizeof (*t);
+  for (t = sh64_media_insn_sem, tend = t + ARRAY_SIZE (sh64_media_insn_sem);
        t != tend; ++t)
     {
       init_idesc (cpu, & table[t->index], t);