Bug 84207 - Hard coded plural in gimple-fold.c points out one of a number of warning_at() calls where warning_n() should have been used. The attached patch both replaces the calls and also changes the signatures of the warning_n(), error_n(), and inform_n() functions to take an unsigned HOST_WIDE_INT argument instead of int. I also changed the implementation of diagnostic_n_impl() to deal with unsigned HOST_WIDE_INT values in excess of ULONG_MAX (the maximum value ngettext handles) so callers don't need to.
Bootstrapped/regtested on x86_64-linux. Martin PS Is there any reason why diagnostic-core.h and diagnostic.c does not/should not include tree.h and other GCC headers?
PR translation/84207 - Hard coded plural in gimple-fold.c gcc/ChangeLog: PR translation/84207 * diagnostic-core.h (warning_n, error_n, inform_n): Change n argument to unsigned HOST_WIDE_INT. * diagnostic.c (warning_n, error_n, inform_n): Ditto. (diagnostic_n_impl): Ditto. Handle arguments in excess of LONG_MAX. * gimple-fold.c (gimple_fold_builtin_strncpy): Use warning_n. Index: gcc/diagnostic-core.h =================================================================== --- gcc/diagnostic-core.h (revision 257607) +++ gcc/diagnostic-core.h (working copy) @@ -59,10 +59,11 @@ extern void internal_error_no_backtrace (const cha ATTRIBUTE_GCC_DIAG(1,2) ATTRIBUTE_NORETURN; /* Pass one of the OPT_W* from options.h as the first parameter. */ extern bool warning (int, const char *, ...) ATTRIBUTE_GCC_DIAG(2,3); -extern bool warning_n (location_t, int, int, const char *, const char *, ...) +extern bool warning_n (location_t, int, unsigned HOST_WIDE_INT, + const char *, const char *, ...) ATTRIBUTE_GCC_DIAG(4,6) ATTRIBUTE_GCC_DIAG(5,6); -extern bool warning_n (rich_location *, int, int, const char *, - const char *, ...) +extern bool warning_n (rich_location *, int, unsigned HOST_WIDE_INT, + const char *, const char *, ...) ATTRIBUTE_GCC_DIAG(4, 6) ATTRIBUTE_GCC_DIAG(5, 6); extern bool warning_at (location_t, int, const char *, ...) ATTRIBUTE_GCC_DIAG(3,4); @@ -69,7 +70,8 @@ extern bool warning_at (location_t, int, const cha extern bool warning_at (rich_location *, int, const char *, ...) ATTRIBUTE_GCC_DIAG(3,4); extern void error (const char *, ...) ATTRIBUTE_GCC_DIAG(1,2); -extern void error_n (location_t, int, const char *, const char *, ...) +extern void error_n (location_t, unsigned HOST_WIDE_INT, const char *, + const char *, ...) ATTRIBUTE_GCC_DIAG(3,5) ATTRIBUTE_GCC_DIAG(4,5); extern void error_at (location_t, const char *, ...) ATTRIBUTE_GCC_DIAG(2,3); extern void error_at (rich_location *, const char *, ...) @@ -87,7 +89,8 @@ extern bool permerror (rich_location *, const char extern void sorry (const char *, ...) ATTRIBUTE_GCC_DIAG(1,2); extern void inform (location_t, const char *, ...) ATTRIBUTE_GCC_DIAG(2,3); extern void inform (rich_location *, const char *, ...) ATTRIBUTE_GCC_DIAG(2,3); -extern void inform_n (location_t, int, const char *, const char *, ...) +extern void inform_n (location_t, unsigned HOST_WIDE_INT, const char *, + const char *, ...) ATTRIBUTE_GCC_DIAG(3,5) ATTRIBUTE_GCC_DIAG(4,5); extern void verbatim (const char *, ...) ATTRIBUTE_GCC_DIAG(1,2); extern bool emit_diagnostic (diagnostic_t, location_t, int, Index: gcc/diagnostic.c =================================================================== --- gcc/diagnostic.c (revision 257607) +++ gcc/diagnostic.c (working copy) @@ -51,8 +51,8 @@ along with GCC; see the file COPYING3. If not see /* Prototypes. */ static bool diagnostic_impl (rich_location *, int, const char *, va_list *, diagnostic_t) ATTRIBUTE_GCC_DIAG(3,0); -static bool diagnostic_n_impl (rich_location *, int, int, const char *, - const char *, va_list *, +static bool diagnostic_n_impl (rich_location *, int, unsigned HOST_WIDE_INT, + const char *, const char *, va_list *, diagnostic_t) ATTRIBUTE_GCC_DIAG(5,0); static void error_recursion (diagnostic_context *) ATTRIBUTE_NORETURN; @@ -1111,15 +1111,22 @@ diagnostic_impl (rich_location *richloc, int opt, /* Implement inform_n, warning_n, and error_n, as documented and defined below. */ static bool -diagnostic_n_impl (rich_location *richloc, int opt, int n, +diagnostic_n_impl (rich_location *richloc, int opt, unsigned HOST_WIDE_INT n, const char *singular_gmsgid, const char *plural_gmsgid, va_list *ap, diagnostic_t kind) { diagnostic_info diagnostic; - diagnostic_set_info_translated (&diagnostic, - ngettext (singular_gmsgid, plural_gmsgid, n), - ap, richloc, kind); + unsigned long gtn; + + if (sizeof n <= sizeof gtn) + gtn = n; + else + /* Use the largest number ngettext() can handle. */ + gtn = n <= ULONG_MAX ? n : ULONG_MAX; + + const char *text = ngettext (singular_gmsgid, plural_gmsgid, gtn); + diagnostic_set_info_translated (&diagnostic, text, ap, richloc, kind); if (kind == DK_WARNING) diagnostic.option_index = opt; return diagnostic_report_diagnostic (global_dc, &diagnostic); @@ -1176,8 +1183,8 @@ inform (rich_location *richloc, const char *gmsgid /* An informative note at LOCATION. Use this for additional details on an error message. */ void -inform_n (location_t location, int n, const char *singular_gmsgid, - const char *plural_gmsgid, ...) +inform_n (location_t location, unsigned HOST_WIDE_INT n, + const char *singular_gmsgid, const char *plural_gmsgid, ...) { va_list ap; va_start (ap, plural_gmsgid); @@ -1233,7 +1240,7 @@ warning_at (rich_location *richloc, int opt, const /* Same as warning_n plural variant below, but using RICHLOC. */ bool -warning_n (rich_location *richloc, int opt, int n, +warning_n (rich_location *richloc, int opt, unsigned HOST_WIDE_INT n, const char *singular_gmsgid, const char *plural_gmsgid, ...) { gcc_assert (richloc); @@ -1252,8 +1259,8 @@ bool Returns true if the warning was printed, false if it was inhibited. */ bool -warning_n (location_t location, int opt, int n, const char *singular_gmsgid, - const char *plural_gmsgid, ...) +warning_n (location_t location, int opt, unsigned HOST_WIDE_INT n, + const char *singular_gmsgid, const char *plural_gmsgid, ...) { va_list ap; va_start (ap, plural_gmsgid); @@ -1350,8 +1357,8 @@ error (const char *gmsgid, ...) /* A hard error: the code is definitely ill-formed, and an object file will not be produced. */ void -error_n (location_t location, int n, const char *singular_gmsgid, - const char *plural_gmsgid, ...) +error_n (location_t location, unsigned HOST_WIDE_INT n, + const char *singular_gmsgid, const char *plural_gmsgid, ...) { va_list ap; va_start (ap, plural_gmsgid); Index: gcc/gimple-fold.c =================================================================== --- gcc/gimple-fold.c (revision 257607) +++ gcc/gimple-fold.c (working copy) @@ -1694,13 +1694,13 @@ gimple_fold_builtin_strncpy (gimple_stmt_iterator tree fndecl = gimple_call_fndecl (stmt); gcall *call = as_a <gcall *> (stmt); - warning_at (loc, OPT_Wstringop_truncation, - (tree_int_cst_equal (size_one_node, len) - ? G_("%G%qD output truncated copying %E byte " - "from a string of length %E") - : G_("%G%qD output truncated copying %E bytes " - "from a string of length %E")), - call, fndecl, len, slen); + warning_n (loc, OPT_Wstringop_truncation, + tree_to_uhwi (len), + "%G%qD output truncated copying %E byte " + "from a string of length %E", + "%G%qD output truncated copying %E bytes " + "from a string of length %E", + call, fndecl, len, slen); } else if (tree_int_cst_equal (len, slen)) { @@ -1707,15 +1707,15 @@ gimple_fold_builtin_strncpy (gimple_stmt_iterator tree fndecl = gimple_call_fndecl (stmt); gcall *call = as_a <gcall *> (stmt); - warning_at (loc, OPT_Wstringop_truncation, - (tree_int_cst_equal (size_one_node, len) - ? G_("%G%qD output truncated before terminating nul " - "copying %E byte from a string of the same " - "length") - : G_("%G%qD output truncated before terminating nul " - "copying %E bytes from a string of the same " - "length")), - call, fndecl, len); + warning_n (loc, OPT_Wstringop_truncation, + tree_to_uhwi (len), + "%G%qD output truncated before terminating nul " + "copying %E byte from a string of the same " + "length", + "%G%qD output truncated before terminating nul " + "copying %E bytes from a string of the same " + "length", + call, fndecl, len); } }