Re: [PATCH] fix ILP32 bootstrap failures due to -Wformat-length

[Jeff Law]

On 09/21/2016 06:28 PM, Martin Sebor wrote:

PR target/77676 - powerpc64 and powerpc64le stage2 bootstrap fail

gcc/testsuite/ChangeLog:
2016-09-21  Martin Sebor  <mse...@redhat.com>

        PR target/77676
        * gcc.dg/tree-ssa/builtin-sprintf-2.c: Fix typo.
        * gcc.dg/tree-ssa/builtin-sprintf-3.c: New test.
        * gcc.dg/tree-ssa/builtin-sprintf-warn-5.c: New test.

gcc/ChangeLog:
2016-09-21  Martin Sebor  <mse...@redhat.com>

        PR target/77676
        * gimple-ssa-sprintf.c (target_int_min, target_int_max): Use
        HOST_BITS_PER_WIDE_INT, make a static local variable auto.
        (target_int_min): Correct computation.
        (format_integer): Use long long as the argument for the ll length
        modifier.
        (format_floating): Use target_int_max().
        (get_string_length): Same.
        (format_string): Avoid setting the bounded flag for strings
        of unknown length.
        (try_substitute_return_value): Avoid setting range info when
        the result isn't bounded.
        * varasm.c (assemble_name): Increase buffer size.

@@ -1471,6 +1473,10 @@ format_string (const conversion_spec &spec, tree arg)
        : 2 == warn_format_length ? 0 <= prec ? prec : 1
        : HOST_WIDE_INT_MAX);

+  /* The result is bounded unless overriddden for a non-constant string

s/overriddden/overridden/

OK with that change.

jeff