From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1851) id 27CF9385AC23; Thu, 13 Jan 2022 17:59:25 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 27CF9385AC23 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Martin Liska To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/marxin/heads/use-ARRAY_SIZE)] Use more ARRAY_SIZE. X-Act-Checkin: gcc X-Git-Author: Martin Liska X-Git-Refname: refs/users/marxin/heads/use-ARRAY_SIZE X-Git-Oldrev: c3782843badbf301da2ed768f5cc13d0e6c453ad X-Git-Newrev: 81fe381150bb36e0bb013cdefe57b086f379cb5d Message-Id: <20220113175925.27CF9385AC23@sourceware.org> Date: Thu, 13 Jan 2022 17:59:25 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 13 Jan 2022 17:59:25 -0000 https://gcc.gnu.org/g:81fe381150bb36e0bb013cdefe57b086f379cb5d commit 81fe381150bb36e0bb013cdefe57b086f379cb5d Author: Martin Liska Date: Thu Jan 13 18:46:26 2022 +0100 Use more ARRAY_SIZE. Diff: --- gcc/ada/locales.c | 6 +++--- gcc/analyzer/engine.cc | 2 +- gcc/analyzer/function-set.cc | 2 +- gcc/analyzer/sm-file.cc | 3 +-- gcc/analyzer/sm-malloc.cc | 3 +-- gcc/analyzer/sm-signal.cc | 3 +-- gcc/attribs.c | 4 ++-- gcc/builtins.c | 2 +- gcc/c-family/c-attribs.c | 3 +-- gcc/c-family/c-common.c | 7 ++----- gcc/c-family/c-format.c | 12 ++++++------ gcc/c-family/c-pragma.c | 9 ++++----- gcc/c-family/c-warn.c | 2 +- gcc/c-family/known-headers.cc | 5 ++--- gcc/c/c-decl.c | 8 +++----- gcc/cfg.c | 2 +- gcc/common/config/aarch64/aarch64-common.c | 3 +-- gcc/config/aarch64/aarch64-builtins.c | 6 +++--- gcc/config/aarch64/aarch64.c | 2 +- gcc/config/arm/arm-builtins.c | 4 ++-- gcc/config/avr/gen-avr-mmcu-texi.c | 6 +++--- gcc/config/c6x/c6x.c | 2 +- gcc/config/darwin-c.c | 2 +- gcc/config/gcn/mkoffload.c | 2 +- gcc/config/i386/i386-builtins.c | 6 ++---- gcc/config/m32c/m32c.c | 2 +- gcc/config/nvptx/mkoffload.c | 6 +++--- gcc/config/rs6000/driver-rs6000.c | 2 +- gcc/config/rs6000/rs6000.c | 2 +- gcc/config/s390/s390.c | 2 +- gcc/config/tilepro/gen-mul-tables.cc | 4 ++-- gcc/config/vms/vms.c | 2 +- gcc/cp/module.cc | 2 +- gcc/cp/name-lookup.c | 2 +- gcc/cp/parser.c | 2 +- gcc/d/longdouble.h | 2 +- gcc/diagnostic-format-json.cc | 2 +- gcc/dwarf2out.c | 3 +-- gcc/fortran/frontend-passes.c | 4 +--- gcc/fortran/openmp.c | 3 +-- gcc/fortran/trans-intrinsic.c | 2 +- gcc/fortran/trans-types.c | 2 +- gcc/genautomata.c | 4 ++-- gcc/genhooks.c | 9 ++++----- gcc/gimple-ssa-sprintf.c | 2 +- gcc/gimple-ssa-warn-access.cc | 2 +- gcc/godump.c | 2 +- gcc/hash-table.c | 2 +- gcc/input.c | 3 +-- gcc/ipa-free-lang-data.cc | 4 +--- gcc/ipa-inline.c | 2 +- gcc/jit/jit-builtins.c | 6 ++---- gcc/jit/jit-recording.c | 4 +--- gcc/lto/lto-common.c | 3 +-- gcc/lto/lto-lang.c | 4 +--- gcc/optc-save-gen.awk | 16 ++++++++-------- gcc/spellcheck.c | 2 +- gcc/sreal.c | 4 ++-- gcc/tree-vect-slp-patterns.c | 2 +- gcc/tree.c | 4 +--- 60 files changed, 97 insertions(+), 128 deletions(-) diff --git a/gcc/ada/locales.c b/gcc/ada/locales.c index ee1385a6bdc..2aaa2121a50 100644 --- a/gcc/ada/locales.c +++ b/gcc/ada/locales.c @@ -649,7 +649,7 @@ str_get_last_byte (char *lc_all) { static char* iso_639_1_to_639_3(char* iso_639_1_code) { - int len = sizeof(iso_639)/sizeof(iso_639[0]); + int len = ARRAY_SIZE (iso_639); char **p = iso_639; int j; @@ -673,7 +673,7 @@ iso_639_1_to_639_3(char* iso_639_1_code) { static char* language_name_to_639_3(char* name) { - int len = sizeof(iso_639)/sizeof(iso_639[0]); + int len = ARRAY_SIZE (iso_639); char **p = iso_639; int j; @@ -695,7 +695,7 @@ language_name_to_639_3(char* name) { static char* country_name_to_3166 (char* name) { - int len = sizeof(iso_3166)/sizeof(iso_3166[0]); + int len = ARRAY_SIZE (iso_3166); char **p = iso_3166; int j; diff --git a/gcc/analyzer/engine.cc b/gcc/analyzer/engine.cc index 8b6f4c83f0f..80ce8a5c701 100644 --- a/gcc/analyzer/engine.cc +++ b/gcc/analyzer/engine.cc @@ -1080,7 +1080,7 @@ exploded_node::get_dot_fillcolor () const = {"azure", "coral", "cornsilk", "lightblue", "yellow", "honeydew", "lightpink", "lightsalmon", "palegreen1", "wheat", "seashell"}; - const int num_colors = sizeof (colors) / sizeof (colors[0]); + const int num_colors = ARRAY_SIZE (colors); return colors[total_sm_state % num_colors]; } else diff --git a/gcc/analyzer/function-set.cc b/gcc/analyzer/function-set.cc index 8d9237deb63..be04186ed0d 100644 --- a/gcc/analyzer/function-set.cc +++ b/gcc/analyzer/function-set.cc @@ -166,7 +166,7 @@ test_stdio_example () "getwc_unlocked", "putc_unlocked" }; - const size_t count = sizeof(example) / sizeof (example[0]); + const size_t count = ARRAY_SIZE (example); function_set fs (example, count); fs.assert_sorted (); fs.assert_sane (); diff --git a/gcc/analyzer/sm-file.cc b/gcc/analyzer/sm-file.cc index 1c996f946f6..5940cb939e5 100644 --- a/gcc/analyzer/sm-file.cc +++ b/gcc/analyzer/sm-file.cc @@ -319,8 +319,7 @@ get_file_using_fns () "ungetc", "vfprintf" }; - const size_t count - = sizeof(funcnames) / sizeof (funcnames[0]); + const size_t count = ARRAY_SIZE (funcnames); function_set fs (funcnames, count); return fs; } diff --git a/gcc/analyzer/sm-malloc.cc b/gcc/analyzer/sm-malloc.cc index 47184969ffe..cc808a6bcf2 100644 --- a/gcc/analyzer/sm-malloc.cc +++ b/gcc/analyzer/sm-malloc.cc @@ -2006,8 +2006,7 @@ malloc_state_machine::unaffected_by_call_p (tree fndecl) /* This array must be kept sorted. */ "strsep", }; - const size_t count - = sizeof(funcnames) / sizeof (funcnames[0]); + const size_t count = ARRAY_SIZE (funcnames); function_set fs (funcnames, count); if (fs.contains_decl_p (fndecl)) diff --git a/gcc/analyzer/sm-signal.cc b/gcc/analyzer/sm-signal.cc index 8d3e5ff07b1..3abfa7db605 100644 --- a/gcc/analyzer/sm-signal.cc +++ b/gcc/analyzer/sm-signal.cc @@ -305,8 +305,7 @@ get_async_signal_unsafe_fns () "vsnprintf", "vsprintf" }; - const size_t count - = sizeof(async_signal_unsafe_fns) / sizeof (async_signal_unsafe_fns[0]); + const size_t count = ARRAY_SIZE (async_signal_unsafe_fns); function_set fs (async_signal_unsafe_fns, count); return fs; } diff --git a/gcc/attribs.c b/gcc/attribs.c index 189f56a6e48..986e2be4027 100644 --- a/gcc/attribs.c +++ b/gcc/attribs.c @@ -499,7 +499,7 @@ diag_attr_exclusions (tree last_decl, tree node, tree attrname, /* Iterate over the mutually exclusive attribute names and verify that the symbol doesn't contain it. */ - for (unsigned i = 0; i != sizeof attrs / sizeof *attrs; ++i) + for (unsigned i = 0; i != ARRAY_SIZE (attrs); ++i) { if (!attrs[i]) continue; @@ -2101,7 +2101,7 @@ decls_mismatched_attributes (tree tmpl, tree decl, tree attrlist, }; for (unsigned i = 0; i != 2; ++i) - for (unsigned j = 0; j != sizeof whitelist / sizeof *whitelist; ++j) + for (unsigned j = 0; j != ARRAY_SIZE (whitelist); ++j) if (lookup_attribute (whitelist[j], tmpl_attrs[i]) || lookup_attribute (whitelist[j], decl_attrs[i])) return 0; diff --git a/gcc/builtins.c b/gcc/builtins.c index c780340ed32..f8869cf3808 100644 --- a/gcc/builtins.c +++ b/gcc/builtins.c @@ -610,7 +610,7 @@ c_strlen (tree arg, int only_value, c_strlen_data *data, unsigned eltsize) if (eltsize != tree_to_uhwi (TYPE_SIZE_UNIT (TREE_TYPE (TREE_TYPE (src))))) return NULL_TREE; - /* Set MAXELTS to sizeof (SRC) / sizeof (*SRC) - 1, the maximum possible + /* Set MAXELTS to ARRAY_SIZE (SRC) - 1, the maximum possible length of SRC. Prefer TYPE_SIZE() to TREE_STRING_LENGTH() if possible in case the latter is less than the size of the array, such as when SRC refers to a short string literal used to initialize a large array. diff --git a/gcc/c-family/c-attribs.c b/gcc/c-family/c-attribs.c index bdf72ce385c..1679820bbe9 100644 --- a/gcc/c-family/c-attribs.c +++ b/gcc/c-family/c-attribs.c @@ -4886,8 +4886,7 @@ handle_access_attribute (tree node[3], tree name, tree args, int flags, int imode; { - const int nmodes = - sizeof attr_access::mode_names / sizeof *attr_access::mode_names; + const int nmodes = ARRAY_SIZE (attr_access::mode_names); for (imode = 0; imode != nmodes; ++imode) if (!strncmp (ps, attr_access::mode_names[imode], diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c index a34f32f51a4..594016c12d8 100644 --- a/gcc/c-family/c-common.c +++ b/gcc/c-family/c-common.c @@ -602,8 +602,7 @@ const struct c_common_resword c_common_reswords[] = { "null_resettable", RID_NULL_RESETTABLE, D_OBJC }, }; -const unsigned int num_c_common_reswords = - sizeof c_common_reswords / sizeof (struct c_common_resword); +const unsigned int num_c_common_reswords = ARRAY_SIZE (c_common_reswords); /* Return identifier for address space AS. */ @@ -4476,9 +4475,7 @@ c_common_nodes_and_builtins (void) /* Make fileptr_type_node a distinct void * type until FILE type is defined. Likewise for const struct tm*. */ - for (unsigned i = 0; - i < sizeof (builtin_structptr_types) / sizeof (builtin_structptr_type); - ++i) + for (unsigned i = 0; i < ARRAY_SIZE (builtin_structptr_types); ++i) builtin_structptr_types[i].node = build_variant_type_copy (builtin_structptr_types[i].base); diff --git a/gcc/c-family/c-format.c b/gcc/c-family/c-format.c index afa77810a5c..52d0b7f6bc8 100644 --- a/gcc/c-family/c-format.c +++ b/gcc/c-family/c-format.c @@ -3195,7 +3195,7 @@ check_tokens (const token_t *tokens, unsigned ntoks, else { /* Diagnose some common misspellings. */ - for (unsigned i = 0; i != sizeof badwords / sizeof *badwords; ++i) + for (unsigned i = 0; i != ARRAY_SIZE (badwords); ++i) { unsigned badwlen = strspn (badwords[i].name, " -"); if (wlen >= badwlen @@ -3390,14 +3390,14 @@ check_plain (location_t format_string_loc, tree format_string_cst, if (ISPUNCT (format_chars[0])) { - size_t nelts = sizeof c_opers / sizeof *c_opers; + size_t nelts = ARRAY_SIZE (c_opers); if (const char *ret = check_tokens (c_opers, nelts, format_string_loc, format_string_cst, orig_format_chars, format_chars, baltoks)) return ret; - nelts = c_dialect_cxx () ? sizeof cxx_opers / sizeof *cxx_opers : 0; + nelts = c_dialect_cxx () ? ARRAY_SIZE (cxx_opers) : 0; if (const char *ret = check_tokens (cxx_opers, nelts, format_string_loc, format_string_cst, orig_format_chars, format_chars, @@ -3407,14 +3407,14 @@ check_plain (location_t format_string_loc, tree format_string_cst, if (ISALPHA (format_chars[0])) { - size_t nelts = sizeof c_keywords / sizeof *c_keywords; + size_t nelts = ARRAY_SIZE (c_keywords); if (const char *ret = check_tokens (c_keywords, nelts, format_string_loc, format_string_cst, orig_format_chars, format_chars, baltoks)) return ret; - nelts = c_dialect_cxx () ? sizeof cxx_keywords / sizeof *cxx_keywords : 0; + nelts = c_dialect_cxx () ? ARRAY_SIZE (cxx_keywords) : 0; if (const char *ret = check_tokens (cxx_keywords, nelts, format_string_loc, format_string_cst, orig_format_chars, format_chars, @@ -3533,7 +3533,7 @@ check_plain (location_t format_string_loc, tree format_string_cst, && ISALPHA (format_chars[1])) { /* Diagnose a subset of contractions that are best avoided. */ - for (unsigned i = 0; i != sizeof contrs / sizeof *contrs; ++i) + for (unsigned i = 0; i != ARRAY_SIZE (contrs); ++i) { const char *apos = strchr (contrs[i].name, '\''); gcc_assert (apos != NULL); diff --git a/gcc/c-family/c-pragma.c b/gcc/c-family/c-pragma.c index 8dde5b841b2..69eda9582a8 100644 --- a/gcc/c-family/c-pragma.c +++ b/gcc/c-family/c-pragma.c @@ -1400,8 +1400,8 @@ static const struct omp_pragma_def omp_pragmas_simd[] = { void c_pp_lookup_pragma (unsigned int id, const char **space, const char **name) { - const int n_oacc_pragmas = sizeof (oacc_pragmas) / sizeof (*oacc_pragmas); - const int n_omp_pragmas = sizeof (omp_pragmas) / sizeof (*omp_pragmas); + const int n_oacc_pragmas = ARRAY_SIZE (oacc_pragmas); + const int n_omp_pragmas = ARRAY_SIZE (omp_pragmas); const int n_omp_pragmas_simd = sizeof (omp_pragmas_simd) / sizeof (*omp_pragmas); int i; @@ -1576,8 +1576,7 @@ init_pragma (void) { if (flag_openacc) { - const int n_oacc_pragmas - = sizeof (oacc_pragmas) / sizeof (*oacc_pragmas); + const int n_oacc_pragmas = ARRAY_SIZE (oacc_pragmas); int i; for (i = 0; i < n_oacc_pragmas; ++i) @@ -1587,7 +1586,7 @@ init_pragma (void) if (flag_openmp) { - const int n_omp_pragmas = sizeof (omp_pragmas) / sizeof (*omp_pragmas); + const int n_omp_pragmas = ARRAY_SIZE (omp_pragmas); int i; for (i = 0; i < n_omp_pragmas; ++i) diff --git a/gcc/c-family/c-warn.c b/gcc/c-family/c-warn.c index 1ce2202c7ff..796912140e3 100644 --- a/gcc/c-family/c-warn.c +++ b/gcc/c-family/c-warn.c @@ -3712,7 +3712,7 @@ warn_parm_array_mismatch (location_t origloc, tree fndecl, tree newparms) to an array and the divisor does not equal the size of the array element. For instance: - sizeof (ARR) / sizeof (OP) + ARRAY_SIZE (ARR) ARR is the array argument of the first sizeof, ARR_TYPE is its ARRAY_TYPE. OP1 is the whole second SIZEOF_EXPR, or its argument; TYPE1 is the type diff --git a/gcc/c-family/known-headers.cc b/gcc/c-family/known-headers.cc index 8ad3eb3f466..01c86b27dc8 100644 --- a/gcc/c-family/known-headers.cc +++ b/gcc/c-family/known-headers.cc @@ -79,8 +79,7 @@ get_string_macro_hint (const char *name, enum stdlib lib) if ((lib == STDLIB_C && flag_isoc99) || (lib == STDLIB_CPLUSPLUS && cxx_dialect >= cxx11 )) { - const size_t num_c99_cxx11_macros - = sizeof (c99_cxx11_macros) / sizeof (c99_cxx11_macros[0]); + const size_t num_c99_cxx11_macros = ARRAY_SIZE (c99_cxx11_macros); for (size_t i = 0; i < num_c99_cxx11_macros; i++) if (strcmp (name, c99_cxx11_macros[i]) == 0) return lib == STDLIB_C ? "" : ""; @@ -204,7 +203,7 @@ get_stdlib_header_for_name (const char *name, enum stdlib lib) {"WCHAR_MAX", {"", ""} }, {"WCHAR_MIN", {"", ""} } }; - const size_t num_hints = sizeof (hints) / sizeof (hints[0]); + const size_t num_hints = ARRAY_SIZE (hints); for (size_t i = 0; i < num_hints; i++) if (strcmp (name, hints[i].name) == 0) return hints[i].header[lib]; diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c index 29a79eba841..c681174c2d3 100644 --- a/gcc/c/c-decl.c +++ b/gcc/c/c-decl.c @@ -1658,7 +1658,7 @@ c_bind (location_t loc, tree decl, bool is_global) Used only by match_builtin_function_types. */ static const unsigned builtin_structptr_type_count - = sizeof builtin_structptr_types / sizeof builtin_structptr_types[0]; + = ARRAY_SIZE (builtin_structptr_types); static GTY(()) tree last_structptr_types[builtin_structptr_type_count]; @@ -1705,10 +1705,8 @@ match_builtin_function_types (tree newtype, tree oldtype, tree newargs = TYPE_ARG_TYPES (newtype); tree tryargs = newargs; - const unsigned nlst - = sizeof last_structptr_types / sizeof last_structptr_types[0]; - const unsigned nbst - = sizeof builtin_structptr_types / sizeof builtin_structptr_types[0]; + const unsigned nlst = ARRAY_SIZE (last_structptr_types); + const unsigned nbst = ARRAY_SIZE (builtin_structptr_types); gcc_checking_assert (nlst == nbst); diff --git a/gcc/cfg.c b/gcc/cfg.c index e4ce360bd87..59c48f3baee 100644 --- a/gcc/cfg.c +++ b/gcc/cfg.c @@ -783,7 +783,7 @@ dump_bb_info (FILE *outf, basic_block bb, int indent, dump_flags_t flags, NULL #undef DEF_BASIC_BLOCK_FLAG }; - const unsigned n_bitnames = sizeof (bb_bitnames) / sizeof (char *); + const unsigned n_bitnames = ARRAY_SIZE (bb_bitnames); bool first; char *s_indent = (char *) alloca ((size_t) indent + 1); memset ((void *) s_indent, ' ', (size_t) indent); diff --git a/gcc/common/config/aarch64/aarch64-common.c b/gcc/common/config/aarch64/aarch64-common.c index 140e3e78d09..6d17b0e133d 100644 --- a/gcc/common/config/aarch64/aarch64-common.c +++ b/gcc/common/config/aarch64/aarch64-common.c @@ -314,8 +314,7 @@ aarch64_option_init_struct (struct gcc_options *opts ATTRIBUTE_UNUSED) pop and attribute change (arm_neon headers, lto etc all cause this to happen quite frequently). It is a trade-off between time and space and so time won. */ - int n_extensions - = sizeof (all_extensions) / sizeof (struct aarch64_option_extension); + int n_extensions = ARRAY_SIZE (all_extensions); qsort (&all_extensions_by_on, n_extensions, sizeof (struct aarch64_option_extension), opt_ext_cmp); } diff --git a/gcc/config/aarch64/aarch64-builtins.c b/gcc/config/aarch64/aarch64-builtins.c index 58bcbd9875f..2d2798602cc 100644 --- a/gcc/config/aarch64/aarch64-builtins.c +++ b/gcc/config/aarch64/aarch64-builtins.c @@ -831,7 +831,7 @@ aarch64_lookup_simd_builtin_type (machine_mode mode, enum aarch64_type_qualifiers q) { int i; - int nelts = sizeof (aarch64_simd_types) / sizeof (aarch64_simd_types[0]); + int nelts = ARRAY_SIZE (aarch64_simd_types); /* Non-poly scalar modes map to standard types not in the table. */ if (q != qualifier_poly && !VECTOR_MODE_P (mode)) @@ -868,7 +868,7 @@ static void aarch64_init_simd_builtin_types (void) { int i; - int nelts = sizeof (aarch64_simd_types) / sizeof (aarch64_simd_types[0]); + int nelts = ARRAY_SIZE (aarch64_simd_types); tree tdecl; /* Init all the element types built by the front-end. */ @@ -1411,7 +1411,7 @@ aarch64_init_builtin_rsqrt (void) }; builtin_decls_data *bdd = bdda; - builtin_decls_data *bdd_end = bdd + (sizeof (bdda) / sizeof (builtin_decls_data)); + builtin_decls_data *bdd_end = bdd + (ARRAY_SIZE (bdda)); for (; bdd < bdd_end; bdd++) { diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index 1bca2a31a1b..9c2259282f4 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -19782,7 +19782,7 @@ is_madd_op (enum attr_type t1) TYPE_SMMLA, TYPE_UMLAL, TYPE_UMLALS,TYPE_SMLSD, TYPE_SMLSDX, TYPE_SMLSLD }; - for (i = 0; i < sizeof (mlatypes) / sizeof (enum attr_type); i++) + for (i = 0; i < ARRAY_SIZE (mlatypes); i++) { if (t1 == mlatypes[i]) return true; diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c index e6bbda23e3e..05bd527e666 100644 --- a/gcc/config/arm/arm-builtins.c +++ b/gcc/config/arm/arm-builtins.c @@ -1445,7 +1445,7 @@ arm_lookup_simd_builtin_type (machine_mode mode, enum arm_type_qualifiers q) { int i; - int nelts = sizeof (arm_simd_types) / sizeof (arm_simd_types[0]); + int nelts = ARRAY_SIZE (arm_simd_types); /* Non-poly scalar modes map to standard types not in the table. */ if (q != qualifier_poly && !VECTOR_MODE_P (mode)) @@ -1477,7 +1477,7 @@ static void arm_init_simd_builtin_types (void) { int i; - int nelts = sizeof (arm_simd_types) / sizeof (arm_simd_types[0]); + int nelts = ARRAY_SIZE (arm_simd_types); tree tdecl; /* Poly types are a world of their own. In order to maintain legacy diff --git a/gcc/config/avr/gen-avr-mmcu-texi.c b/gcc/config/avr/gen-avr-mmcu-texi.c index 05b81dd3cb5..34fb3943a62 100644 --- a/gcc/config/avr/gen-avr-mmcu-texi.c +++ b/gcc/config/avr/gen-avr-mmcu-texi.c @@ -26,7 +26,7 @@ #include "avr-devices.c" static const avr_mcu_t* -mcus[sizeof avr_mcu_types / sizeof avr_mcu_types[0]]; +mcus[ARRAY_SIZE (avr_mcu_types); static int letter (char c) { @@ -56,7 +56,7 @@ c_prefix (const char *str) "attiny", "atmega", "atxmega", "ata", "at90" }; - int i, n = (int) (sizeof (prefixes) / sizeof (*prefixes)); + int i, n = (int) (ARRAY_SIZE (prefixes)); for (i = 0; i < n; i++) if (str_prefix_p (str, prefixes[i])) @@ -185,7 +185,7 @@ int main (void) print_mcus (n_mcus); n_mcus = 0; - for (i = 0; i < sizeof (avr_texinfo) / sizeof (*avr_texinfo); i++) + for (i = 0; i < ARRAY_SIZE (avr_texinfo); i++) if (arch_id == avr_texinfo[i].arch_id) printf ("@item %s\n%s\n", mcu->name, avr_texinfo[i].texinfo); } diff --git a/gcc/config/c6x/c6x.c b/gcc/config/c6x/c6x.c index 7fe18d69c7f..dc01a6e47c4 100644 --- a/gcc/config/c6x/c6x.c +++ b/gcc/config/c6x/c6x.c @@ -2580,7 +2580,7 @@ static unsigned reg_save_order[] = REG_B14, REG_A15 }; -#define N_SAVE_ORDER (sizeof reg_save_order / sizeof *reg_save_order) +#define N_SAVE_ORDER (ARRAY_SIZE (reg_save_order)) /* Compute the layout of the stack frame and store it in FRAME. */ diff --git a/gcc/config/darwin-c.c b/gcc/config/darwin-c.c index 9f46924408a..58deb01405a 100644 --- a/gcc/config/darwin-c.c +++ b/gcc/config/darwin-c.c @@ -505,7 +505,7 @@ darwin_register_frameworks (const char *sysroot, size_t i; /* Setup default search path for frameworks. */ - for (i=0; i> CHAR_BIT * i); } diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c index 056002e4a4a..760eb687d23 100644 --- a/gcc/config/s390/s390.c +++ b/gcc/config/s390/s390.c @@ -8774,7 +8774,7 @@ static machine_mode constant_modes[] = QImode, V1QImode }; -#define NR_C_MODES (sizeof (constant_modes) / sizeof (constant_modes[0])) +#define NR_C_MODES (ARRAY_SIZE (constant_modes)) struct constant { diff --git a/gcc/config/tilepro/gen-mul-tables.cc b/gcc/config/tilepro/gen-mul-tables.cc index c9649fb9c70..798766a723b 100644 --- a/gcc/config/tilepro/gen-mul-tables.cc +++ b/gcc/config/tilepro/gen-mul-tables.cc @@ -462,7 +462,7 @@ find_sequences (ExpressionTree &s, ExpressionTreeMap &best_solution) const Operator *const prev_op = s.m_exprs[num_vals - 1].m_op; const int prev_top_index = (prev_op != NULL) ? prev_op->m_top_index : -1; - for (size_t f = 0; f < sizeof ops / sizeof ops[0]; f++) + for (size_t f = 0; f < ARRAY_SIZE (ops); f++) { const Operator *const op = &ops[f]; @@ -564,7 +564,7 @@ create_insn_code_compression_table () printf ("const enum insn_code %s_multiply_insn_seq_decode_opcode[] = {\n" " CODE_FOR_nothing /* must be first */ ", ARCH); - for (size_t i = 0; i < sizeof ops / sizeof ops[0]; i++) + for (size_t i = 0; i < ARRAY_SIZE (ops); i++) { Operator *op = &ops[i]; int index = -1; diff --git a/gcc/config/vms/vms.c b/gcc/config/vms/vms.c index 5d565e35fb4..d0d44bad2ce 100644 --- a/gcc/config/vms/vms.c +++ b/gcc/config/vms/vms.c @@ -99,7 +99,7 @@ static const struct vms_crtl_name vms_crtl_names[] = /* Number of entires in the above array. */ -#define NBR_CRTL_NAMES (sizeof (vms_crtl_names) / sizeof (*vms_crtl_names)) +#define NBR_CRTL_NAMES (ARRAY_SIZE (vms_crtl_names)) /* List of aliased identifiers. They must be persistent across gc. */ diff --git a/gcc/cp/module.cc b/gcc/cp/module.cc index 01341301811..125896a82e8 100644 --- a/gcc/cp/module.cc +++ b/gcc/cp/module.cc @@ -2615,7 +2615,7 @@ depset::entity_kind_name () const {"decl", "specialization", "partial", "using", "namespace", "redirect", "binding"}; entity_kind kind = get_entity_kind (); - gcc_checking_assert (kind < sizeof (names) / sizeof(names[0])); + gcc_checking_assert (kind < ARRAY_SIZE (names)); return names[kind]; } diff --git a/gcc/cp/name-lookup.c b/gcc/cp/name-lookup.c index caa23d3e6f8..cf68c70a0d7 100644 --- a/gcc/cp/name-lookup.c +++ b/gcc/cp/name-lookup.c @@ -6880,7 +6880,7 @@ get_std_name_hint (const char *name) /* . */ {"vector", "", cxx98}, }; - const size_t num_hints = sizeof (hints) / sizeof (hints[0]); + const size_t num_hints = ARRAY_SIZE (hints); for (size_t i = 0; i < num_hints; i++) { if (strcmp (name, hints[i].name) == 0) diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c index f40e707d47c..9f1a69be4a0 100644 --- a/gcc/cp/parser.c +++ b/gcc/cp/parser.c @@ -4178,7 +4178,7 @@ cp_parser_new (cp_lexer *lexer) { /* Initialize the binops_by_token so that we can get the tree directly from the token. */ - for (unsigned i = 0; i < sizeof (binops) / sizeof (binops[0]); i++) + for (unsigned i = 0; i < ARRAY_SIZE (binops); i++) binops_by_token[binops[i].token_type] = binops[i]; cp_parser *parser = ggc_cleared_alloc (); diff --git a/gcc/d/longdouble.h b/gcc/d/longdouble.h index 1e457ae04d6..2d9695a4309 100644 --- a/gcc/d/longdouble.h +++ b/gcc/d/longdouble.h @@ -117,7 +117,7 @@ public: private: /* Including gcc/real.h presents too many problems, so just statically allocate enough space for REAL_VALUE_TYPE. */ - long realvalue[(2 + (16 + sizeof (long)) / sizeof (long))]; + long realvalue[(2 + (16 + ARRAY_SIZE (long))]; }; /* Declared, but "volatile" is not required. */ diff --git a/gcc/diagnostic-format-json.cc b/gcc/diagnostic-format-json.cc index def371816e6..62594ebb4d7 100644 --- a/gcc/diagnostic-format-json.cc +++ b/gcc/diagnostic-format-json.cc @@ -62,7 +62,7 @@ json_from_expanded_location (diagnostic_context *context, location_t loc) {"byte-column", DIAGNOSTICS_COLUMN_UNIT_BYTE} }; int the_column = INT_MIN; - for (int i = 0; i != sizeof column_fields / sizeof (*column_fields); ++i) + for (int i = 0; i != ARRAY_SIZE (column_fields); ++i) { context->column_unit = column_fields[i].unit; const int col = diagnostic_converted_column (context, exploc); diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index 094fd0acdb0..8031896ea3b 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -13536,8 +13536,7 @@ static const dwarf_qual_info_t dwarf_qual_info[] = { TYPE_QUAL_RESTRICT, DW_TAG_restrict_type }, { TYPE_QUAL_ATOMIC, DW_TAG_atomic_type } }; -static const unsigned int dwarf_qual_info_size - = sizeof (dwarf_qual_info) / sizeof (dwarf_qual_info[0]); +static const unsigned int dwarf_qual_info_size = ARRAY_SIZE (dwarf_qual_info); /* If DIE is a qualified DIE of some base DIE with the same parent, return the base DIE, otherwise return NULL. Set MASK to the diff --git a/gcc/fortran/frontend-passes.c b/gcc/fortran/frontend-passes.c index 22f1bb56a2b..907d15cbfe4 100644 --- a/gcc/fortran/frontend-passes.c +++ b/gcc/fortran/frontend-passes.c @@ -5653,9 +5653,7 @@ gfc_code_walker (gfc_code **c, walk_code_fn_t codefn, walk_expr_fn_t exprfn, WALK_SUBEXPR (co->ext.omp_clauses->detach); for (idx = 0; idx < OMP_IF_LAST; idx++) WALK_SUBEXPR (co->ext.omp_clauses->if_exprs[idx]); - for (idx = 0; - idx < sizeof (list_types) / sizeof (list_types[0]); - idx++) + for (idx = 0; idx < ARRAY_SIZE (list_types); idx++) for (n = co->ext.omp_clauses->lists[list_types[idx]]; n; n = n->next) WALK_SUBEXPR (n->expr); diff --git a/gcc/fortran/openmp.c b/gcc/fortran/openmp.c index 86c412a4334..c2fe1d0d783 100644 --- a/gcc/fortran/openmp.c +++ b/gcc/fortran/openmp.c @@ -4864,8 +4864,7 @@ gfc_match_omp_context_selector_specification (gfc_omp_declare_variant *odv) match m; const char *selector_sets[] = { "construct", "device", "implementation", "user" }; - const int selector_set_count - = sizeof (selector_sets) / sizeof (*selector_sets); + const int selector_set_count = ARRAY_SIZE (selector_sets); int i; char buf[GFC_MAX_SYMBOL_LEN + 1]; diff --git a/gcc/fortran/trans-intrinsic.c b/gcc/fortran/trans-intrinsic.c index dc72d994781..946853c5b37 100644 --- a/gcc/fortran/trans-intrinsic.c +++ b/gcc/fortran/trans-intrinsic.c @@ -9770,7 +9770,7 @@ conv_intrinsic_ieee_builtin (gfc_se * se, gfc_expr * expr, enum built_in_function code, int nargs) { tree args[2]; - gcc_assert ((unsigned) nargs <= sizeof(args)/sizeof(args[0])); + gcc_assert ((unsigned) nargs <= ARRAY_SIZE (args)); conv_ieee_function_args (se, expr, args, nargs); se->expr = build_call_expr_loc_array (input_location, diff --git a/gcc/fortran/trans-types.c b/gcc/fortran/trans-types.c index 5de43bb2784..5dc6064e186 100644 --- a/gcc/fortran/trans-types.c +++ b/gcc/fortran/trans-types.c @@ -3417,7 +3417,7 @@ gfc_get_array_descr_info (const_tree type, struct array_descr_info *info) } rank = GFC_TYPE_ARRAY_RANK (type); - if (rank >= (int) (sizeof (info->dimen) / sizeof (info->dimen[0]))) + if (rank >= (int) (ARRAY_SIZE (info->dimen))) return false; etype = GFC_TYPE_ARRAY_DATAPTR_TYPE (type); diff --git a/gcc/genautomata.c b/gcc/genautomata.c index 5872aca88fa..1d5be8841c5 100644 --- a/gcc/genautomata.c +++ b/gcc/genautomata.c @@ -8623,8 +8623,8 @@ output_get_cpu_unit_code_func (void) fprintf (output_file, " };\n\n"); fprintf (output_file, " /* The following is binary search: */\n"); fprintf (output_file, " %s = 0;\n", LOW_VARIABLE_NAME); - fprintf (output_file, " %s = sizeof (%s) / sizeof (struct %s) - 1;\n", - HIGH_VARIABLE_NAME, NAME_CODE_TABLE_NAME, NAME_CODE_STRUCT_NAME); + fprintf (output_file, " %s = ARRAY_SIZE (%s) - 1;\n", + HIGH_VARIABLE_NAME, NAME_CODE_TABLE_NAME); fprintf (output_file, " while (%s <= %s)\n {\n", LOW_VARIABLE_NAME, HIGH_VARIABLE_NAME); fprintf (output_file, " %s = (%s + %s) / 2;\n", diff --git a/gcc/genhooks.c b/gcc/genhooks.c index 20ad607b158..6bae85d7b8d 100644 --- a/gcc/genhooks.c +++ b/gcc/genhooks.c @@ -128,7 +128,7 @@ emit_documentation (const char *in_fname) } fclose (f); /* For each hook in hook_array, if it is a start hook, store its position. */ - for (i = 0; i < (int) (sizeof hook_array / sizeof hook_array[0]); i++) + for (i = 0; i < (int) (ARRAY_SIZE (hook_array)); i++) { struct s_hook sh, *shp; void *p; @@ -223,7 +223,7 @@ emit_documentation (const char *in_fname) /* POD-valued hooks sometimes come in groups with common documentation.*/ for (j = i + 1; - j < (int) (sizeof hook_array / sizeof hook_array[0]) + j < (int) (ARRAY_SIZE (hook_array)) && hook_array[j].doc == 0 && hook_array[j].type; j++) { char *namex = upstrdup (hook_array[j].name); @@ -246,8 +246,7 @@ emit_documentation (const char *in_fname) printf ("\n@end %s", deftype); } } - if (++i >= (int) (sizeof hook_array / sizeof hook_array[0]) - || !hook_array[i].doc) + if (++i >= (int) (ARRAY_SIZE (hook_array)) || !hook_array[i].doc) break; free (name); sh.name = name = upstrdup (hook_array[i].name); @@ -270,7 +269,7 @@ emit_init_macros (const char *docname) for (print_nest = 0; print_nest <= MAX_NEST; print_nest++) { - for (i = 0; i < (int) (sizeof hook_array / sizeof hook_array[0]); i++) + for (i = 0; i < (int) (ARRAY_SIZE (hook_array)); i++) { char *name = upstrdup (hook_array[i].name); diff --git a/gcc/gimple-ssa-sprintf.c b/gcc/gimple-ssa-sprintf.c index 1f756b74ca9..21b2408bc3e 100644 --- a/gcc/gimple-ssa-sprintf.c +++ b/gcc/gimple-ssa-sprintf.c @@ -1953,7 +1953,7 @@ format_floating (const directive &dir, tree arg, range_query *) &res.range.min, &res.range.max }; - for (int i = 0; i != sizeof minmax / sizeof *minmax; ++i) + for (int i = 0; i != ARRAY_SIZE (minmax); ++i) { /* Convert the GCC real value representation with the precision of the real type to the mpfr_t format rounding down in the diff --git a/gcc/gimple-ssa-warn-access.cc b/gcc/gimple-ssa-warn-access.cc index e4c078a8e43..efaf19b800e 100644 --- a/gcc/gimple-ssa-warn-access.cc +++ b/gcc/gimple-ssa-warn-access.cc @@ -2773,7 +2773,7 @@ memmodel_name (unsigned HOST_WIDE_INT val) { val = memmodel_base (val); - for (unsigned i = 0; i != sizeof memory_models / sizeof *memory_models; ++i) + for (unsigned i = 0; i != ARRAY_SIZE (memory_models); ++i) { if (val == memory_models[i].modval) return memory_models[i].modname; diff --git a/gcc/godump.c b/gcc/godump.c index 2092446b0cc..9af1f9d9cf4 100644 --- a/gcc/godump.c +++ b/gcc/godump.c @@ -1327,7 +1327,7 @@ static void keyword_hash_init (class godump_container *container) { size_t i; - size_t count = sizeof (keywords) / sizeof (keywords[0]); + size_t count = ARRAY_SIZE (keywords); void **slot; for (i = 0; i < count; i++) diff --git a/gcc/hash-table.c b/gcc/hash-table.c index dad1d21f099..1015c1ee6e9 100644 --- a/gcc/hash-table.c +++ b/gcc/hash-table.c @@ -84,7 +84,7 @@ unsigned int hash_table_higher_prime_index (unsigned long n) { unsigned int low = 0; - unsigned int high = sizeof (prime_tab) / sizeof (prime_tab[0]); + unsigned int high = ARRAY_SIZE (prime_tab); while (low != high) { diff --git a/gcc/input.c b/gcc/input.c index 82e79beab14..0db28462d62 100644 --- a/gcc/input.c +++ b/gcc/input.c @@ -3724,8 +3724,7 @@ for_each_line_table_case (void (*testcase) (const line_table_case &)) { /* ...and use each of the "interesting" location values as the starting location within line_table. */ - const int num_boundary_locations - = sizeof (boundary_locations) / sizeof (boundary_locations[0]); + const int num_boundary_locations = ARRAY_SIZE (boundary_locations); for (int loc_idx = 0; loc_idx < num_boundary_locations; loc_idx++) { line_table_case c (default_range_bits, boundary_locations[loc_idx]); diff --git a/gcc/ipa-free-lang-data.cc b/gcc/ipa-free-lang-data.cc index 431ced4dd17..0520e16a0f2 100644 --- a/gcc/ipa-free-lang-data.cc +++ b/gcc/ipa-free-lang-data.cc @@ -1109,9 +1109,7 @@ free_lang_data (void) free_lang_data_in_cgraph (&fld); /* Create gimple variants for common types. */ - for (unsigned i = 0; - i < sizeof (builtin_structptr_types) / sizeof (builtin_structptr_type); - ++i) + for (unsigned i = 0; i < ARRAY_SIZE (builtin_structptr_types); ++i) builtin_structptr_types[i].node = builtin_structptr_types[i].base; /* Reset some langhooks. Do not reset types_compatible_p, it may diff --git a/gcc/ipa-inline.c b/gcc/ipa-inline.c index 9b70daf8976..2aae6ea0a0f 100644 --- a/gcc/ipa-inline.c +++ b/gcc/ipa-inline.c @@ -278,7 +278,7 @@ sanitize_attrs_match_for_inline_p (const_tree caller, const_tree callee) SANITIZE_POINTER_SUBTRACT }; - for (unsigned i = 0; i < sizeof (codes) / sizeof (codes[0]); i++) + for (unsigned i = 0; i < ARRAY_SIZE (codes); i++) if (sanitize_flags_p (codes[i], caller) != sanitize_flags_p (codes[i], callee)) return false; diff --git a/gcc/jit/jit-builtins.c b/gcc/jit/jit-builtins.c index 0b6210807be..a95f9213780 100644 --- a/gcc/jit/jit-builtins.c +++ b/gcc/jit/jit-builtins.c @@ -109,9 +109,7 @@ find_builtin_by_name (const char *in_name, We start at index 1 to skip the initial entry (BUILT_IN_NONE), which has a NULL name. */ - for (unsigned int i = 1; - i < sizeof (builtin_data) / sizeof (builtin_data[0]); - i++) + for (unsigned int i = 1; i < ARRAY_SIZE (builtin_data); i++) { const struct builtin_data& bd = builtin_data[i]; if (matches_builtin (in_name, bd)) @@ -320,7 +318,7 @@ static const char * const type_names[] = { static const char * get_string_for_type_id (enum jit_builtin_type type_id) { - gcc_assert (type_id < sizeof (type_names)/sizeof(type_names[0])); + gcc_assert (type_id < ARRAY_SIZE (type_names)); return type_names[type_id]; } diff --git a/gcc/jit/jit-recording.c b/gcc/jit/jit-recording.c index ee8934131d1..10ca636e60f 100644 --- a/gcc/jit/jit-recording.c +++ b/gcc/jit/jit-recording.c @@ -568,9 +568,7 @@ recording::context::context (context *parent_ctxt) if (parent_ctxt) { /* Inherit options from parent. */ - for (unsigned i = 0; - i < sizeof (m_str_options) / sizeof (m_str_options[0]); - i++) + for (unsigned i = 0; i < ARRAY_SIZE (m_str_options); i++) { const char *parent_opt = parent_ctxt->m_str_options[i]; m_str_options[i] = parent_opt ? xstrdup (parent_opt) : NULL; diff --git a/gcc/lto/lto-common.c b/gcc/lto/lto-common.c index 853b3fbd856..8b0572b1283 100644 --- a/gcc/lto/lto-common.c +++ b/gcc/lto/lto-common.c @@ -2101,8 +2101,7 @@ lto_resolution_read (splay_tree file_ids, FILE *resolution, lto_file *file) char r_str[27]; enum ld_plugin_symbol_resolution r = (enum ld_plugin_symbol_resolution) 0; unsigned int j; - unsigned int lto_resolution_str_len - = sizeof (lto_resolution_str) / sizeof (char *); + unsigned int lto_resolution_str_len = ARRAY_SIZE (lto_resolution_str); res_pair rp; t = fscanf (resolution, "%u " HOST_WIDE_INT_PRINT_HEX_PURE diff --git a/gcc/lto/lto-lang.c b/gcc/lto/lto-lang.c index a7f6f02c79c..4148bb96211 100644 --- a/gcc/lto/lto-lang.c +++ b/gcc/lto/lto-lang.c @@ -1319,9 +1319,7 @@ lto_init (void) distinction should only be relevant to the front-end, so we always use the C definition here in lto1. Likewise for const struct tm*. */ - for (unsigned i = 0; - i < sizeof (builtin_structptr_types) / sizeof (builtin_structptr_type); - ++i) + for (unsigned i = 0; i < ARRAY_SIZE (builtin_structptr_types); ++i) { gcc_assert (builtin_structptr_types[i].node == builtin_structptr_types[i].base); diff --git a/gcc/optc-save-gen.awk b/gcc/optc-save-gen.awk index 34261a7ef44..7e996783bff 100644 --- a/gcc/optc-save-gen.awk +++ b/gcc/optc-save-gen.awk @@ -1104,7 +1104,7 @@ for (i = 0; i < n_target_val; i++) { } if (has_target_explicit_mask) { - print " for (size_t i = 0; i < sizeof (ptr1->explicit_mask) / sizeof (ptr1->explicit_mask[0]); i++)"; + print " for (size_t i = 0; i < ARRAY_SIZE (ptr1->explicit_mask); i++)"; print " if (ptr1->explicit_mask[i] != ptr2->explicit_mask[i])"; print " return false;" } @@ -1152,7 +1152,7 @@ for (i = 0; i < n_target_val; i++) { print " hstate.add_hwi (ptr->" name");"; } if (has_target_explicit_mask) { - print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; + print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " hstate.add_hwi (ptr->explicit_mask[i]);"; } @@ -1192,7 +1192,7 @@ for (i = 0; i < n_target_val; i++) { } if (has_target_explicit_mask) { - print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; + print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " bp_pack_value (bp, ptr->explicit_mask[i], 64);"; } @@ -1235,7 +1235,7 @@ for (i = 0; i < n_target_val; i++) { } if (has_target_explicit_mask) { - print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; + print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " ptr->explicit_mask[i] = bp_unpack_value (bp, 64);"; } @@ -1317,7 +1317,7 @@ for (i = 0; i < n_opt_val; i++) { else print " hstate.add_hwi (ptr->" name");"; } -print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; +print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " hstate.add_hwi (ptr->explicit_mask[i]);"; print " return hstate.end ();"; print "}"; @@ -1346,7 +1346,7 @@ for (i = 0; i < n_opt_val; i++) { print " return false;"; } } -print " for (size_t i = 0; i < sizeof (ptr1->explicit_mask) / sizeof (ptr1->explicit_mask[0]); i++)"; +print " for (size_t i = 0; i < ARRAY_SIZE (ptr1->explicit_mask); i++)"; print " if (ptr1->explicit_mask[i] != ptr2->explicit_mask[i])"; print " return false;" print " return true;"; @@ -1380,7 +1380,7 @@ for (i = 0; i < n_opt_val; i++) { } } } -print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; +print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " bp_pack_value (bp, ptr->explicit_mask[i], 64);"; print "}"; @@ -1412,7 +1412,7 @@ for (i = 0; i < n_opt_val; i++) { } } } -print " for (size_t i = 0; i < sizeof (ptr->explicit_mask) / sizeof (ptr->explicit_mask[0]); i++)"; +print " for (size_t i = 0; i < ARRAY_SIZE (ptr->explicit_mask); i++)"; print " ptr->explicit_mask[i] = bp_unpack_value (bp, 64);"; print "}"; print "/* Free heap memory used by optimization options */"; diff --git a/gcc/spellcheck.c b/gcc/spellcheck.c index 6077bed7861..5875c10a96f 100644 --- a/gcc/spellcheck.c +++ b/gcc/spellcheck.c @@ -489,7 +489,7 @@ static const char * const test_data[] = { static void test_metric_conditions () { - const int num_test_cases = sizeof (test_data) / sizeof (test_data[0]); + const int num_test_cases = ARRAY_SIZE (test_data); for (int i = 0; i < num_test_cases; i++) { diff --git a/gcc/sreal.c b/gcc/sreal.c index 960f0da6fd7..6dd23b27bb5 100644 --- a/gcc/sreal.c +++ b/gcc/sreal.c @@ -323,7 +323,7 @@ static void sreal_verify_arithmetics (void) { int values[] = {-14123413, -7777, -17, -10, -2, 0, 17, 139, 1234123}; - unsigned c = sizeof (values) / sizeof (int); + unsigned c = ARRAY_SIZE (values); for (unsigned i = 0; i < c; i++) for (unsigned j = 0; j < c; j++) @@ -359,7 +359,7 @@ static void sreal_verify_shifting (void) { int values[] = {0, 17, 32, 139, 1024, 55555, 1234123}; - unsigned c = sizeof (values) / sizeof (int); + unsigned c = ARRAY_SIZE (values); for (unsigned i = 0; i < c; i++) verify_shifting (values[i]); diff --git a/gcc/tree-vect-slp-patterns.c b/gcc/tree-vect-slp-patterns.c index 63fe75606ab..86b22941d0f 100644 --- a/gcc/tree-vect-slp-patterns.c +++ b/gcc/tree-vect-slp-patterns.c @@ -1584,4 +1584,4 @@ vect_pattern_decl_t slp_patterns[] #undef SLP_PATTERN /* Set the number of SLP pattern matchers available. */ -size_t num__slp_patterns = sizeof(slp_patterns)/sizeof(vect_pattern_decl_t); +size_t num__slp_patterns = ARRAY_SIZE (slp_patterns); diff --git a/gcc/tree.c b/gcc/tree.c index d98b77db50b..8489f586381 100644 --- a/gcc/tree.c +++ b/gcc/tree.c @@ -9295,9 +9295,7 @@ build_common_tree_nodes (bool signed_char) ptr_type_node = build_pointer_type (void_type_node); const_ptr_type_node = build_pointer_type (build_type_variant (void_type_node, 1, 0)); - for (unsigned i = 0; - i < sizeof (builtin_structptr_types) / sizeof (builtin_structptr_type); - ++i) + for (unsigned i = 0; i < ARRAY_SIZE (builtin_structptr_types); ++i) builtin_structptr_types[i].node = builtin_structptr_types[i].base; pointer_sized_int_node = build_nonstandard_integer_type (POINTER_SIZE, 1);