Re: [PATCH] Don't segv in omp-low.c (PR middle-end/59669)

2014-01-08 Thread Jakub Jelinek
On Wed, Jan 08, 2014 at 04:25:47PM +0100, Marek Polacek wrote: > Indeed it does. So like this? > > 2014-01-08 Marek Polacek > > PR middle-end/59669 > * omp-low.c (simd_clone_adjust): Don't crash if def is NULL. > testsuite/ > * gcc.dg/gomp/pr59669-1.c: New test. > * gc

Re: [PATCH] Don't segv in omp-low.c (PR middle-end/59669)

2014-01-08 Thread Marek Polacek
On Wed, Jan 08, 2014 at 04:14:06PM +0100, Jakub Jelinek wrote: > On Wed, Jan 08, 2014 at 04:09:08PM +0100, Marek Polacek wrote: > > We can also get NULL for the default definition, so we need to handle that > > before calling has_zero_uses on it. > > > > Bootstrapped/regtested on x86_64-linux, ok

Re: [PATCH] Don't segv in omp-low.c (PR middle-end/59669)

2014-01-08 Thread Jakub Jelinek
On Wed, Jan 08, 2014 at 04:09:08PM +0100, Marek Polacek wrote: > We can also get NULL for the default definition, so we need to handle that > before calling has_zero_uses on it. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? Looks ok, but there is similar code a few lines above, can yo

[PATCH] Don't segv in omp-low.c (PR middle-end/59669)

2014-01-08 Thread Marek Polacek
We can also get NULL for the default definition, so we need to handle that before calling has_zero_uses on it. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2014-01-08 Marek Polacek PR middle-end/59669 * omp-low.c (simd_clone_adjust): Don't crash if def is NULL. testsu