Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-05-02 Thread Tom de Vries
On 05/01/2017 08:05 PM, Martin Sebor wrote: On 04/30/2017 02:02 PM, Tom de Vries wrote: On 01/10/2017 11:16 PM, Martin Sebor wrote: + __builtin_sprintf (d, "%32s", "x"); /* { dg-warning "directive writing 32 bytes into a region of size 12" "-Wformat-length" { xfail *-*-* } } */ This

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-05-01 Thread Martin Sebor
On 04/30/2017 02:02 PM, Tom de Vries wrote: On 01/10/2017 11:16 PM, Martin Sebor wrote: + __builtin_sprintf (d, "%32s", "x"); /* { dg-warning "directive writing 32 bytes into a region of size 12" "-Wformat-length" { xfail *-*-* } } */ This xpasses for me on an older system: ... XPASS:

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-04-30 Thread Tom de Vries
On 01/10/2017 11:16 PM, Martin Sebor wrote: + __builtin_sprintf (d, "%32s", "x"); /* { dg-warning "directive writing 32 bytes into a region of size 12" "-Wformat-length" { xfail *-*-* } } */ This xpasses for me on an older system: ... XPASS: gcc.dg/pr78768.c -Wformat-overflow (test for

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-17 Thread Martin Sebor
On 01/17/2017 05:04 AM, Kyrill Tkachov wrote: Hi Martin, On 10/01/17 22:16, Martin Sebor wrote: The -Walloca-larger-than, -Wformat-length, and -Wformat-truncation options do not mention LTO among the supported languages and so are disabled when -flto is used, causing false negatives. The

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-17 Thread Kyrill Tkachov
Hi Martin, On 10/01/17 22:16, Martin Sebor wrote: The -Walloca-larger-than, -Wformat-length, and -Wformat-truncation options do not mention LTO among the supported languages and so are disabled when -flto is used, causing false negatives. The attached patch adds the missing LTO to the three

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-13 Thread Andreas Schwab
On Jan 10 2017, Martin Sebor wrote: > Index: gcc/testsuite/gcc.dg/pr78768.c > === > --- gcc/testsuite/gcc.dg/pr78768.c(revision 0) > +++ gcc/testsuite/gcc.dg/pr78768.c(working copy) > @@ -0,0 +1,13 @@ > +/*

Re: [PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-11 Thread Richard Biener
On Tue, 10 Jan 2017, Martin Sebor wrote: > The -Walloca-larger-than, -Wformat-length, and -Wformat-truncation > options do not mention LTO among the supported languages and so are > disabled when -flto is used, causing false negatives. > > The attached patch adds the missing LTO to the three

[PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-10 Thread Martin Sebor
The -Walloca-larger-than, -Wformat-length, and -Wformat-truncation options do not mention LTO among the supported languages and so are disabled when -flto is used, causing false negatives. The attached patch adds the missing LTO to the three options. This makes -Walloca-larger-than work with