Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 07:55:26PM +, Zamyatin, Igor wrote: > So below is the patch for 4.9 trunk. Ok, thanks. > > Thanks, > Igor > > gcc/c/Changelog: > > 2014-04-10 Igor Zamyatin > > PR middle-end/60469 > * c-array-notation.c (fix_builtin_array_notation_fn): Use > cr

RE: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
> -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Thursday, April 10, 2014 10:21 PM > To: Zamyatin, Igor > Cc: GCC Patches (gcc-patches@gcc.gnu.org); Iyer, Balaji V > Subject: Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array &

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 06:16:22PM +, Zamyatin, Igor wrote: > Fixed patch is below. > > Just out of curiosity - why do you think this patch is not for 4.9? The one with just create_tmp_var (integer_type_node, NULL); is for 4.9. For the iterator var changes, I guess one needs to analyze all th

RE: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
fine ALEN 1024 + +int main(int argc, char* argv[]) +{ + int b[ALEN]; + b[:] = 100; + _Cilk_spawn foo(); + return 0; + > -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Thursday, April 10, 2014 9:29 PM > To: Zamyatin, Igor > Cc: GCC Patches (g

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 05:19:33PM +, Zamyatin, Igor wrote: > @@ -282,8 +283,7 @@ fix_builtin_array_notation_fn (tree an_builtin_fn, tree > *new_var) > >for (ii = 0; ii < rank; ii++) > { > - an_loop_info[ii].var = build_decl (location, VAR_DECL, NULL_TREE, > -

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Rainer Orth
"Zamyatin, Igor" writes: > diff --git a/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c > b/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c > new file mode 100644 > index 000..61e8014 > --- /dev/null > +++ b/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c > @@ -0,0 +1,15 @@ > +/* middl

[PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
Hi! This patches fixes the ICE when array notation is used along with Cilk_spawn. We need to expand AN's loop using other routine for creating temporary variables. Bootstrapped/regtested on x86_64. Also it introduces no new failures and fixes couple of ICE for CilkPlus Conformance suite which