Re: [PATCH] handle local aggregate initialization in strlen, take 2 (PR 83821)

[Jakub Jelinek]

On Mon, Oct 14, 2019 at 06:23:22PM -0600, Martin Sebor wrote:
> 
> gcc/ChangeLog:
> 
>       PR tree-optimization/83821
>       * tree-ssa-strlen.c (maybe_invalidate): Add argument.  Consider
>       the length of a string when available.

> +                   fprintf (dump_file,
> +                            "  statement may clobber string %zu long\n",
> +                            tree_to_uhwi (size));
> +                 else
> +                   fprintf (dump_file,
> +                            "  statement may clobber string\n");

This broke bootstrap on i686-linux and likely many other hosts.

I'll commit the following as obvious if it passes bootstrap.

2019-10-16  Jakub Jelinek  <ja...@redhat.com>

        * tree-ssa-strlen.c (maybe_invalidate): Use
        HOST_WIDE_INT_PRINT_UNSIGNED instead of "%zu".

--- gcc/tree-ssa-strlen.c.jj    2019-10-16 23:10:18.641727500 +0200
+++ gcc/tree-ssa-strlen.c       2019-10-16 23:44:43.202636942 +0200
@@ -1130,7 +1130,8 @@ maybe_invalidate (gimple *stmt, bool zer
                  {
                    if (size && tree_fits_uhwi_p (size))
                      fprintf (dump_file,
-                              "  statement may clobber string %zu long\n",
+                              "  statement may clobber string "
+                              HOST_WIDE_INT_PRINT_UNSIGNED " long\n",
                               tree_to_uhwi (size));
                    else
                      fprintf (dump_file,


        Jakub