Re: PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-22 Thread Martin Sebor
On 2/22/19 4:33 PM, Jakub Jelinek wrote: On Fri, Feb 22, 2019 at 04:17:14PM -0700, Martin Sebor wrote: On 2/22/19 3:44 PM, Jakub Jelinek wrote: On Tue, Feb 19, 2019 at 05:43:53PM -0700, Martin Sebor wrote: PR tree-optimization/88993 PR tree-optimization/88835 * gcc.dg/t

Re: PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-22 Thread Jakub Jelinek
On Fri, Feb 22, 2019 at 04:17:14PM -0700, Martin Sebor wrote: > On 2/22/19 3:44 PM, Jakub Jelinek wrote: > > On Tue, Feb 19, 2019 at 05:43:53PM -0700, Martin Sebor wrote: > > > PR tree-optimization/88993 > > > PR tree-optimization/88835 > > > * gcc.dg/tree-ssa/builtin-fprintf-warn-2.c: New te

Re: PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-22 Thread Martin Sebor
On 2/22/19 3:44 PM, Jakub Jelinek wrote: On Tue, Feb 19, 2019 at 05:43:53PM -0700, Martin Sebor wrote: PR tree-optimization/88993 PR tree-optimization/88835 * gcc.dg/tree-ssa/builtin-fprintf-warn-2.c: New test. * gcc.dg/tree-ssa/builtin-printf-warn-2.c: New test.

Re: PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-22 Thread Jakub Jelinek
On Tue, Feb 19, 2019 at 05:43:53PM -0700, Martin Sebor wrote: > PR tree-optimization/88993 > PR tree-optimization/88835 > * gcc.dg/tree-ssa/builtin-fprintf-warn-2.c: New test. > * gcc.dg/tree-ssa/builtin-printf-warn-2.c: New test. > * gcc.dg/tree-ssa/builtin-snprintf-w

Re: PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-21 Thread Jeff Law
On 2/19/19 5:43 PM, Martin Sebor wrote: > I received feedback on the first patch that it doesn't suppress all > the instances of the warning so I've relaxed the checker even more > to avoid the excess instances seen in Elfutils and beefed up > the tests.  The top of trunk compiles cleanly now with 

PING [PATCH] avoid 4095/INT_MAX warning for fprintf (PR 88993)

2019-02-19 Thread Martin Sebor
I received feedback on the first patch that it doesn't suppress all the instances of the warning so I've relaxed the checker even more to avoid the excess instances seen in Elfutils and beefed up the tests. The top of trunk compiles cleanly now with just the three instances of -Wformat-truncation