Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-28 Thread Aldy Hernandez
On 11/28/2016 04:51 PM, Jeff Law wrote: On 11/25/2016 07:02 AM, Andreas Schwab wrote: On Nov 23 2016, Christophe Lyon wrote: Since you committed this patch (r242733), I've noticed a regression: gcc.dg/uninit-pred-6_c.c warning (test for warnings, line 43) now fails on some arm targets, for

Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-28 Thread Jeff Law
On 11/25/2016 07:02 AM, Andreas Schwab wrote: On Nov 23 2016, Christophe Lyon wrote: Since you committed this patch (r242733), I've noticed a regression: gcc.dg/uninit-pred-6_c.c warning (test for warnings, line 43) now fails on some arm targets, for instance arm-none-linux-gnueabihf --with

Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-25 Thread Andreas Schwab
On Nov 23 2016, Christophe Lyon wrote: > Since you committed this patch (r242733), I've noticed a regression: > gcc.dg/uninit-pred-6_c.c warning (test for warnings, line 43) > > now fails on some arm targets, for instance arm-none-linux-gnueabihf > --with-cpu=cortex-a5 --with-fpu=vfpv3-d16-fp16

Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-23 Thread Christophe Lyon
Hi Marek, On 22 November 2016 at 01:02, Marek Polacek wrote: > What seems like a typo caused an ICE here. We've got a vector of vectors here > and we're trying to walk all the elements, so the second loop oughta use 'j'. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? > > 2016-11-21

Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-22 Thread Aldy Hernandez
On 11/22/2016 02:49 AM, Jakub Jelinek wrote: On Mon, Nov 21, 2016 at 04:02:40PM -0800, Marek Polacek wrote: What seems like a typo caused an ICE here. We've got a vector of vectors here and we're trying to walk all the elements, so the second loop oughta use 'j'. Bootstrapped/regtested on x86_

Re: [PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-21 Thread Jakub Jelinek
On Mon, Nov 21, 2016 at 04:02:40PM -0800, Marek Polacek wrote: > What seems like a typo caused an ICE here. We've got a vector of vectors here > and we're trying to walk all the elements, so the second loop oughta use 'j'. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? > > 2016-11-21

[PATCH] Fix ICE with -Wuninitialized (PR tree-optimization/78455)

2016-11-21 Thread Marek Polacek
What seems like a typo caused an ICE here. We've got a vector of vectors here and we're trying to walk all the elements, so the second loop oughta use 'j'. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-11-21 Marek Polacek PR tree-optimization/78455 * tree-ssa-uni