]> git.ipfire.org Git - thirdparty/gcc.git/commit
Fix up dump_printf_loc format attribute and adjust uses [PR106782]
authorJakub Jelinek <jakub@redhat.com>
Thu, 1 Sep 2022 09:07:44 +0000 (11:07 +0200)
committerJakub Jelinek <jakub@redhat.com>
Thu, 1 Sep 2022 09:38:46 +0000 (11:38 +0200)
commitf9593025a290c68c0916dc6fa569eb38eda00535
treeb4f7c01042ac51e112a5406453014635d85538cd
parentb33602e5b834177f202422b3b4191af9fd79d2d2
Fix up dump_printf_loc format attribute and adjust uses [PR106782]

As discussed on IRC, the r13-2299-g68c61c2daa1f bug only got missed
because dump_printf_loc had incorrect format attribute and therefore
almost no -Wformat=* checking was performed on it.
3, 0 are suitable for function with (whatever, whatever, const char *, va_list)
arguments, not for (whatever, whatever, const char *, ...), that one should
use 3, 4.

There are 3 spots where the mismatch was worse though, two using %u or %d
for unsigned HOST_WIDE_INT argument and one %T for enum argument (promoted
to int) and this backport just fixes those spots.

2022-09-01  Jakub Jelinek  <jakub@redhat.com>

PR other/106782
* tree-vect-slp.cc (vect_print_slp_tree): Use
HOST_WIDE_INT_PRINT_UNSIGNED instead of %u.
* tree-vect-loop.cc (vect_estimate_min_profitable_iters): Use
HOST_WIDE_INT_PRINT_UNSIGNED instead of %d.
* tree-vect-slp-patterns.cc (vect_pattern_validate_optab): Use %G
instead of %T and STMT_VINFO_STMT (SLP_TREE_REPRESENTATIVE (node))
instead of SLP_TREE_DEF_TYPE (node).

(cherry picked from commit 953e08fde44a596e4ec2491efd15cd645e1ddc48)
gcc/tree-vect-loop.cc
gcc/tree-vect-slp-patterns.cc
gcc/tree-vect-slp.cc