Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-03-04 Thread Jeff Law
On 03/04/15 06:18, Martin Liška wrote: Hello. There is a patch with 2 hunks that are related to the same problem on powerpc64le-unknown-linux-gnu. Ready for trunk? Thanks, Martin 0001-Fix-warnings-occured-during-profiledboostrap-on-powe.patch From ea05ba5d0e4a5963e063564ffbec84f90fac

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-03-04 Thread Martin Liška
On 01/27/2015 01:31 PM, Martin Liška wrote: On 01/27/2015 05:23 AM, DJ Delorie wrote: +/* Workaround -Wstrict-overflow false positive during profiledbootstrap. */ + +# if GCC_VERSION >= 4004 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wstrict-overflow" +#endif + #pragma dia

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-30 Thread Jakub Jelinek
On Fri, Jan 30, 2015 at 12:46:43PM +0100, Martin Liška wrote: > There's a small patch that updates the version. > > Ready for trunk? > 2015-01-30 Martin Liska > > * tree.h: Guard GCC version for a pragma ifdef. * tree.h: Change GCC_VERSION >= 4004 to GCC_VERSION >= 4006

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-30 Thread Martin Liška
On 01/27/2015 01:31 PM, Martin Liška wrote: On 01/27/2015 05:23 AM, DJ Delorie wrote: +/* Workaround -Wstrict-overflow false positive during profiledbootstrap. */ + +# if GCC_VERSION >= 4004 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wstrict-overflow" +#endif + #pragma dia

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-27 Thread Martin Liška
On 01/27/2015 05:23 AM, DJ Delorie wrote: +/* Workaround -Wstrict-overflow false positive during profiledbootstrap. */ + +# if GCC_VERSION >= 4004 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wstrict-overflow" +#endif + #pragma diagnostic ignored was added in 4.4 but #pragma

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-26 Thread DJ Delorie
> +/* Workaround -Wstrict-overflow false positive during profiledbootstrap. */ > + > +# if GCC_VERSION >= 4004 > +#pragma GCC diagnostic push > +#pragma GCC diagnostic ignored "-Wstrict-overflow" > +#endif > + #pragma diagnostic ignored was added in 4.4 but #pragma diagnostic push/pop wasn't add

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-22 Thread Jeff Law
On 01/19/15 11:30, Martin Liška wrote: On 01/19/2015 03:48 PM, Jakub Jelinek wrote: On Mon, Jan 19, 2015 at 03:44:20PM +0100, Martin Liška wrote: There's a bunch of another places that emit false positives. With following patch, I am able to run profiledbootstrap on x86_64-linux-pc and ppc64-li

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-19 Thread Martin Liška
On 01/19/2015 03:48 PM, Jakub Jelinek wrote: On Mon, Jan 19, 2015 at 03:44:20PM +0100, Martin Liška wrote: There's a bunch of another places that emit false positives. With following patch, I am able to run profiledbootstrap on x86_64-linux-pc and ppc64-linux-pc. I don't know what we consider

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-19 Thread Jakub Jelinek
On Mon, Jan 19, 2015 at 03:44:20PM +0100, Martin Liška wrote: > There's a bunch of another places that emit false positives. With following > patch, > I am able to run profiledbootstrap on x86_64-linux-pc and ppc64-linux-pc. I don't know what we consider standard and where we allow warnings and t

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-19 Thread Martin Liška
On 01/15/2015 12:35 PM, Richard Biener wrote: On Thu, Jan 15, 2015 at 12:29 PM, Jakub Jelinek wrote: On Thu, Jan 15, 2015 at 12:17:44PM +0100, Richard Biener wrote: On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek wrote: Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootst

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 12:29 PM, Jakub Jelinek wrote: > On Thu, Jan 15, 2015 at 12:17:44PM +0100, Richard Biener wrote: >> On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek wrote: >> > Hi! >> > >> > I ran into -Werror=maybe-uninitialized errors during profiledbootstrap >> > (../configure --enable-

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Jakub Jelinek
On Thu, Jan 15, 2015 at 12:17:44PM +0100, Richard Biener wrote: > On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek wrote: > > Hi! > > > > I ran into -Werror=maybe-uninitialized errors during profiledbootstrap > > (../configure --enable-languages=c,c++ --enable-checking=release; > > make -j16 profil

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek wrote: > Hi! > > I ran into -Werror=maybe-uninitialized errors during profiledbootstrap > (../configure --enable-languages=c,c++ --enable-checking=release; > make -j16 profiledboostrap) before I hit a miscompilation I'm going to file. > > Is this ok

[PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Jakub Jelinek
Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootstrap (../configure --enable-languages=c,c++ --enable-checking=release; make -j16 profiledboostrap) before I hit a miscompilation I'm going to file. Is this ok for trunk, or do we want to work around them differently? 2015-01-1