gcse, ipa-devirt: Use %wd/%wu instead of HOST_WIDE_INT_PRINT* in diagnostics [PR99288]
HOST_WIDE_INT_PRINT* in the string literals of warning/error/inform etc. make those messages non-translatable, and we have a perfectly fine alternative when not using system *printf - %w{d,u}. 2021-02-27 Jakub Jelinek <jakub@redhat.com> PR other/99288 * gcse.c (gcse_or_cprop_is_too_expensive): Use %wu instead of HOST_WIDE_INT_PRINT_UNSIGNED in warning format string. * ipa-devirt.c (ipa_odr_read_section): Use %wd instead of HOST_WIDE_INT_PRINT_DEC in inform format string. Fix comment typos.
This commit is contained in:
parent
06a9f20f97
commit
652623f7c6
@ -4011,8 +4011,7 @@ gcse_or_cprop_is_too_expensive (const char *pass)
|
||||
{
|
||||
warning (OPT_Wdisabled_optimization,
|
||||
"%s: %d basic blocks and %d registers; "
|
||||
"increase %<--param max-gcse-memory%> above "
|
||||
HOST_WIDE_INT_PRINT_UNSIGNED,
|
||||
"increase %<--param max-gcse-memory%> above %wu",
|
||||
pass, n_basic_blocks_for_fn (cfun), max_reg_num (),
|
||||
memory_request / 1024);
|
||||
|
||||
|
@ -4261,13 +4261,12 @@ ipa_odr_read_section (struct lto_file_decl_data *file_data, const char *data,
|
||||
" in another translation unit",
|
||||
this_enum.vals[j].name, warn_name);
|
||||
/* FIXME: In case there is easy way to print wide_ints,
|
||||
perhaps we could do it here instead of overlfow checpl. */
|
||||
perhaps we could do it here instead of overflow check. */
|
||||
else if (wi::fits_shwi_p (this_enum.vals[j].val)
|
||||
&& wi::fits_shwi_p (warn_value))
|
||||
inform (this_enum.vals[j].locus,
|
||||
"name %qs is defined to " HOST_WIDE_INT_PRINT_DEC
|
||||
" while another translation unit defines "
|
||||
"it as " HOST_WIDE_INT_PRINT_DEC,
|
||||
"name %qs is defined to %wd while another "
|
||||
"translation unit defines it as %wd",
|
||||
warn_name, this_enum.vals[j].val.to_shwi (),
|
||||
warn_value.to_shwi ());
|
||||
else
|
||||
|
Loading…
Reference in New Issue
Block a user